Update E2E test of assigning code owners
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)