Investigate if view_code_coverage_spec can be removed
With our E2E test suite having causing an increased execution time, we're reviewing our E2E tests to ensure that what we run in this test suite are specs that really do require being tested against a live environment.
The view_code_coverage_spec is one such spec we're looking into as a potential spec to remove and be replaced with a combination of lower level tests that would essentially test the same scenarios.
The goal of this issue is as follows:
- Identify existing lower level specs (unit / integration) that overlaps with parts of this E2E test.
- If there is insufficient coverage, identify additional tests we need to introduce at the unit / integration test levels.
Once we've determined that it is possible to cover this test at the lower level, we can then proceed with removing this E2E test which would help improve our pipeline runtime.
Action
- Create new test case in feature spec
spec/features/merge_request/user_sees_merge_widget_spec.rb
: user views MR when pipeline has code coverage enabled MR widget shows the coverage. - Remove E2E test
qa/qa/specs/features/browser_ui/4_verify/testing/view_code_coverage_spec.rb