Skip to content

E2E test coverage review for `qa/specs/features/ee/browser_ui/10_govern/group/group_audit_logs_2_spec.rb`

E2E test coverage review

In #418898 (closed) several E2E tests were identified as possible candidates for replacement by tests at lower levels of the test pyramid.

The aim of this issue is identify existing unit/integration/system/feature tests that also cover the feature tested by the following E2E test:

The spec file includes tests for the following events:

  • Changed lfs enabled
  • Changed membership lock
  • Changed request access enabled
  • Changed require two factor authentication
  • Changed project creation level

There is another issue for a similar test: #427668

Outcome

The result should be a list of tests that cover the functionality essential to the feature, and a decision about whether the E2E test can be removed.

  • If lower-level coverage is complete, notify @mlapierre that the E2E test can be removed.
  • If there are test gaps that would be reasonable to cover at a lower level, add the tests or open another issue so that the test can be added at a later date.
  • If there are test gaps that can't efficiently/effectively be covered at a lower level, this issue can be closed.

Please notify @mlapierre of the outcome, or if there are any questions 😄

Edited by Mark Lapierre