Skip to content

E2E test: Only enable importing for large MRs

Jay McCure requested to merge jmc-fix-mr-admin-specs into master

What does this MR do and why?

Related to #410505 (closed)

Currently the following tests enable importing, even though they only use the Import functionality during the large-setup job. Importing is used to import a large MR during this job. qa/qa/specs/features/browser_ui/3_create/merge_request/revert/reverting_merge_request_spec.rb qa/qa/specs/features/browser_ui/3_create/merge_request/view_merge_request_diff_patch_spec.rb

This moves QA::Support::Helpers::ImportSource.enable(%w[gitlab_project]) to the fabricate_large_merge_request method where it is required. This will make the tests faster and more reliable when run outside of the large-setup job.

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Edited by Jay McCure

Merge request reports