Skip to content

EE: Unify CE and EE tests in spec/controllers/projects/artifacts_controller_spec.rb

Lin Jen-Shin requested to merge 3377-use-artifacts-tests-from-ee-ee into master

What does this MR do?

Mark it as EE specific. I would love to extract this to a separate file, but... for this specific case, it would make the test much harder to understand, and we might need to copy a lot if we're not making shared examples in another file and include it, which would make it harder to understand as well.

Now I miss Gitlab.ee? and feel it would be easier to have it.

Are there points in the code the reviewer needs to double check?

Should we just comment like this?

Does this MR meet the acceptance criteria?

  • Review
    • Has been reviewed by Backend

Merge request reports