Skip to content

Update E2E test of assigning code owners

Mark Lapierre requested to merge ml-qa-fix-merge-request-approvals-test into master

What does this MR do?

A recent change removed the option to require approval from codeowners from project settings and added it to the protected branch creation screen.

See !15862 (merged)

This updates the E2E test of the affected feature:

  • converts the old qa class selectors to data attributes.
  • removes the EE project settings module because it only contained the merge request approval setting, which only include the code owners setting and was removed with the feature change
  • fixes a bug in Resource::Repository::Commit that caused new files to not be committed
  • updates the test to conform with our current style

Closes #33680 (closed)

Does this MR meet the acceptance criteria?

Conformity

Merge request reports