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

See https://gitlab.com/gitlab-org/gitlab/merge_requests/15862

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