Commit 55df1d6b authored by Scott Hampton's avatar Scott Hampton

Fix event and feature flag name

Fix event and feature flag name to include `i` and
`testing`. Also, I originally messed up the event
name altogether so it needed to be fixed anyway.
parent 793161d7
......@@ -44,7 +44,6 @@ class Projects::MergeRequestsController < Projects::MergeRequests::ApplicationCo
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(:usage_data_group_code_coverage_visit_total, @project, default_enabled: true)
record_experiment_user(:invite_members_version_a)
record_experiment_user(:invite_members_version_b)
......
---
name: usage_data_group_code_coverage_visit_total
name: usage_data_i_testing_metrics_report_widget_total
introduced_by_url: https://gitlab.com/gitlab-org/gitlab/-/merge_requests/50790
rollout_issue_url:
milestone: '13.8'
......
......@@ -65,7 +65,7 @@ export default {
},
handleToggleEvent() {
return once(() => {
if (this.glFeatures.usageDataGroupCodeCoverageVisitTotal) {
if (this.glFeatures.usageDataITestingMetricsReportWidgetTotal) {
api.trackRedisHllUserEvent(this.$options.expandEvent);
}
});
......@@ -78,7 +78,7 @@ export default {
methods: {
...mapActions(['setEndpoint', 'fetchMetrics']),
},
expandEvent: 'group_code_coverage_visit_total',
expandEvent: 'i_testing_metrics_report_widget_total',
};
</script>
<template>
......
......@@ -11,6 +11,7 @@ module EE
before_action only: [:show] do
push_frontend_feature_flag(:anonymous_visual_review_feedback)
push_frontend_feature_flag(:missing_mr_security_scan_types, @project)
push_frontend_feature_flag(:usage_data_i_testing_metrics_report_widget_total, @project, default_enabled: true)
end
before_action :whitelist_query_limiting_ee_merge, only: [:merge]
......
......@@ -83,7 +83,7 @@ describe('Grouped metrics reports app', () => {
describe('with :usage_data_group_code_coverage_visit_total enabled', () => {
beforeEach(() => {
mountComponent({ usageDataGroupCodeCoverageVisitTotal: true });
mountComponent({ usageDataITestingMetricsReportWidgetTotal: true });
});
it('tracks group_code_coverage_visit_total metric', () => {
......@@ -96,7 +96,7 @@ describe('Grouped metrics reports app', () => {
describe('with :usage_data_group_code_coverage_visit_total disabled', () => {
beforeEach(() => {
mountComponent({ usageDataGroupCodeCoverageVisitTotal: false });
mountComponent({ usageDataITestingMetricsReportWidgetTotal: false });
});
it('does not track group_code_coverage_visit_total metric', () => {
......
......@@ -248,11 +248,11 @@
redis_slot: testing
aggregation: weekly
feature_flag: usage_data_i_testing_test_case_parsed
- name: group_code_coverage_visit_total
- name: i_testing_metrics_report_widget_total
category: testing
redis_slot: testing
aggregation: weekly
feature_flag: usage_data_group_code_coverage_visit_total
feature_flag: usage_data_i_testing_metrics_report_widget_total
# Project Management group
- name: g_project_management_issue_title_changed
category: issues_edit
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment