Merge branch 'eb-remove-test-failure-history-flag' into 'master'

Remove test_failure_history flag

See merge request gitlab-org/gitlab!51464

app/assets/javascripts/reports/components/grouped_test_reports_app.vue

@@ -10,7 +10,6 @@ import IssuesList from './issues_list.vue';
 import Modal from './modal.vue';
 import createStore from '../store';
 import Tracking from '~/tracking';
-import glFeatureFlagsMixin from '~/vue_shared/mixins/gl_feature_flags_mixin';
 import {
   summaryTextBuilder,
   reportTextBuilder,
@@ -28,7 +27,7 @@ export default {
     Modal,
     GlButton,
   },
-  mixins: [glFeatureFlagsMixin(), Tracking.mixin()],
+  mixins: [Tracking.mixin()],
   props: {
     endpoint: {
       type: String,
@@ -92,7 +91,7 @@ export default {
       return reportTextBuilder(name, summary);
     },
     hasRecentFailures(summary) {
-      return this.glFeatures.testFailureHistory && summary?.recentlyFailed > 0;
+      return summary?.recentlyFailed > 0;
     },
     recentFailuresText(summary) {
       return recentFailuresTextBuilder(summary);

app/assets/javascripts/reports/components/test_issue_body.vue

 <script>
 import { mapActions } from 'vuex';
 import { GlBadge, GlSprintf } from '@gitlab/ui';
-import glFeatureFlagsMixin from '~/vue_shared/mixins/gl_feature_flags_mixin';
 
 export default {
   name: 'TestIssueBody',
@@ -9,7 +8,6 @@ export default {
     GlBadge,
     GlSprintf,
   },
-  mixins: [glFeatureFlagsMixin()],
   props: {
     issue: {
       type: Object,
@@ -28,11 +26,7 @@ export default {
   },
   computed: {
     showRecentFailures() {
-      return (
-        this.glFeatures.testFailureHistory &&
-        this.issue.recent_failures?.count &&
-        this.issue.recent_failures?.base_branch
-      );
+      return this.issue.recent_failures?.count && this.issue.recent_failures?.base_branch;
     },
   },
   methods: {

app/controllers/projects/merge_requests_controller.rb

@@ -39,7 +39,6 @@ class Projects::MergeRequestsController < Projects::MergeRequests::ApplicationCo
     push_frontend_feature_flag(:core_security_mr_widget_counts, @project)
     push_frontend_feature_flag(:core_security_mr_widget_downloads, @project, default_enabled: true)
     push_frontend_feature_flag(:remove_resolve_note, @project, default_enabled: true)
-    push_frontend_feature_flag(:test_failure_history, @project)
     push_frontend_feature_flag(:diffs_gradual_load, @project, default_enabled: true)
     push_frontend_feature_flag(:codequality_mr_diff, @project)
 

app/services/ci/test_failure_history_service.rb

@@ -30,7 +30,6 @@ module Ci
     end
 
     def should_track_failures?
-      return false unless Feature.enabled?(:test_failure_history, project)
       return false unless project.default_branch_or_master == pipeline.ref
 
       # We fetch for up to MAX_TRACKABLE_FAILURES + 1 builds. So if ever we get

changelogs/unreleased/eb-remove-test-failure-history-flag.yml

+---
+title: Remove test_failure_history flag
+merge_request: 51464
+author:
+type: changed

config/feature_flags/development/test_failure_history.yml

----
-name: test_failure_history
-introduced_by_url: https://gitlab.com/gitlab-org/gitlab/-/merge_requests/45027
-rollout_issue_url: https://gitlab.com/gitlab-org/gitlab/-/issues/268249
-milestone: '13.6'
-type: development
-group: group::testing
-default_enabled: false

doc/ci/unit_test_reports.md

@@ -68,36 +68,11 @@ execution time and the error output.
 ### Number of recent failures
 
 > - [Introduced](https://gitlab.com/gitlab-org/gitlab/-/issues/241759) in GitLab 13.7.
-> - It's [deployed behind a feature flag](../user/feature_flags.md), disabled by default.
-> - It's disabled on GitLab.com.
-> - It's not recommended for production use.
-> - To use it in GitLab self-managed instances, ask a GitLab administrator to [enable it](#enable-or-disable-the-number-of-recent-failures). **(CORE ONLY)**
-
-WARNING:
-This feature might not be available to you. Check the **version history** note above for details.
+> - [Feature flag removed](https://gitlab.com/gitlab-org/gitlab/-/issues/268249) in GitLab 13.8.
 
 If a test failed in the project's default branch in the last 14 days, a message like
 `Failed {n} time(s) in {default_branch} in the last 14 days` is displayed for that test.
 
-#### Enable or disable the number of recent failures **(CORE ONLY)**
-
-Displaying the number of failures in the last 14 days is under development and not
-ready for production use. It is deployed behind a feature flag that is **disabled by default**.
-[GitLab administrators with access to the GitLab Rails console](../administration/feature_flags.md)
-can enable it.
-
-To enable it:
-
-```ruby
-Feature.enable(:test_failure_history)
-```
-
-To disable it:
-
-```ruby
-Feature.disable(:test_failure_history)
-```
-
 ## How to set it up
 
 To enable the Unit test reports in merge requests, you need to add

lib/gitlab/ci/reports/test_failure_history.rb

@@ -12,8 +12,6 @@ module Gitlab
         end
 
         def load!
-          return unless Feature.enabled?(:test_failure_history, project)
-
           recent_failures_count.each do |key_hash, count|
             failed_test_cases[key_hash].set_recent_failures(count, project.default_branch_or_master)
           end

spec/frontend/reports/components/grouped_test_reports_app_spec.js

@@ -22,7 +22,7 @@ describe('Grouped test reports app', () => {
   let wrapper;
   let mockStore;
 
-  const mountComponent = ({ props = { pipelinePath }, testFailureHistory = false } = {}) => {
+  const mountComponent = ({ props = { pipelinePath } } = {}) => {
     wrapper = mount(Component, {
       store: mockStore,
       localVue,
@@ -31,11 +31,6 @@ describe('Grouped test reports app', () => {
         pipelinePath,
         ...props,
       },
-      provide: {
-        glFeatures: {
-          testFailureHistory,
-        },
-      },
     });
   };
 
@@ -242,50 +237,25 @@ describe('Grouped test reports app', () => {
     describe('with recent failures counts', () => {
       beforeEach(() => {
         setReports(recentFailuresTestReports);
+        mountComponent();
       });
 
-      describe('with feature flag enabled', () => {
-        beforeEach(() => {
-          mountComponent({ testFailureHistory: true });
-        });
-
-        it('renders the recently failed tests summary', () => {
-          expect(findHeader().text()).toContain(
-            '2 out of 3 failed tests have failed more than once in the last 14 days',
-          );
-        });
-
-        it('renders the recently failed count on the test suite', () => {
-          expect(findSummaryDescription().text()).toContain(
-            '1 out of 2 failed tests has failed more than once in the last 14 days',
-          );
-        });
-
-        it('renders the recent failures count on the test case', () => {
-          expect(findIssueDescription().text()).toContain(
-            'Failed 8 times in master in the last 14 days',
-          );
-        });
+      it('renders the recently failed tests summary', () => {
+        expect(findHeader().text()).toContain(
+          '2 out of 3 failed tests have failed more than once in the last 14 days',
+        );
       });
 
-      describe('with feature flag disabled', () => {
-        beforeEach(() => {
-          mountComponent({ testFailureHistory: false });
-        });
-
-        it('does not render the recently failed tests summary', () => {
-          expect(findHeader().text()).not.toContain('failed more than once in the last 14 days');
-        });
-
-        it('does not render the recently failed count on the test suite', () => {
-          expect(findSummaryDescription().text()).not.toContain(
-            'failed more than once in the last 14 days',
-          );
-        });
+      it('renders the recently failed count on the test suite', () => {
+        expect(findSummaryDescription().text()).toContain(
+          '1 out of 2 failed tests has failed more than once in the last 14 days',
+        );
+      });
 
-        it('renders the recent failures count on the test case', () => {
-          expect(findIssueDescription().text()).not.toContain('in the last 14 days');
-        });
+      it('renders the recent failures count on the test case', () => {
+        expect(findIssueDescription().text()).toContain(
+          'Failed 8 times in master in the last 14 days',
+        );
       });
     });
 

spec/lib/gitlab/ci/reports/test_failure_history_spec.rb

@@ -28,18 +28,5 @@ RSpec.describe Gitlab::Ci::Reports::TestFailureHistory, :aggregate_failures do
       expect(failed_rspec.recent_failures).to eq(count: 2, base_branch: 'master')
       expect(failed_java.recent_failures).to eq(count: 1, base_branch: 'master')
     end
-
-    context 'when feature flag is disabled' do
-      before do
-        stub_feature_flags(test_failure_history: false)
-      end
-
-      it 'does not set recent failures' do
-        load_history
-
-        expect(failed_rspec.recent_failures).to be_nil
-        expect(failed_java.recent_failures).to be_nil
-      end
-    end
   end
 end

spec/services/ci/test_failure_history_service_spec.rb

@@ -22,19 +22,6 @@ RSpec.describe Ci::TestFailureHistoryService, :aggregate_failures do
         expect(Ci::TestCaseFailure.count).to eq(2)
       end
 
-      context 'when feature flag for test failure history is disabled' do
-        before do
-          stub_feature_flags(test_failure_history: false)
-        end
-
-        it 'does not persist data' do
-          execute_service
-
-          expect(Ci::TestCase.count).to eq(0)
-          expect(Ci::TestCaseFailure.count).to eq(0)
-        end
-      end
-
       context 'when pipeline is not for the default branch' do
         before do
           pipeline.update_column(:ref, 'new-feature')
@@ -136,14 +123,6 @@ RSpec.describe Ci::TestFailureHistoryService, :aggregate_failures do
       it { is_expected.to eq(true) }
     end
 
-    context 'when feature flag is disabled' do
-      before do
-        stub_feature_flags(test_failure_history: false)
-      end
-
-      it { is_expected.to eq(false) }
-    end
-
     context 'when pipeline is not equal to the project default branch' do
       before do
         pipeline.update_column(:ref, 'some-other-branch')