Merge branch 'astoicescu/settingsButton' into 'master'

Add a metrics settings button to the dashboard header

See merge request gitlab-org/gitlab!35848

app/assets/javascripts/monitoring/components/dashboard_header.vue

@@ -2,6 +2,7 @@
 import { debounce } from 'lodash';
 import { mapActions, mapState, mapGetters } from 'vuex';
 import {
+  GlButton,
   GlIcon,
   GlDeprecatedButton,
   GlDropdown,
@@ -37,6 +38,7 @@ import { timezones } from '../format_date';
 export default {
   components: {
     Icon,
+    GlButton,
     GlIcon,
     GlDeprecatedButton,
     GlDropdown,
@@ -123,6 +125,8 @@ export default {
       'showEmptyState',
       'dashboardTimezone',
       'projectPath',
+      'canAccessOperationsSettings',
+      'operationsSettingsPath',
     ]),
     ...mapGetters('monitoringDashboard', ['selectedDashboard', 'filteredEnvironments']),
     isSystemDashboard() {
@@ -150,6 +154,9 @@ export default {
     shouldShowActionsMenu() {
       return Boolean(this.projectPath);
     },
+    shouldShowSettingsButton() {
+      return this.canAccessOperationsSettings && this.operationsSettingsPath;
+    },
   },
   methods: {
     ...mapActions('monitoringDashboard', ['filterEnvironments', 'toggleStarredValue']),
@@ -381,10 +388,14 @@ export default {
         </gl-deprecated-button>
       </div>
 
-      <template v-if="shouldShowActionsMenu">
-        <span aria-hidden="true" class="gl-pl-3 border-left gl-mb-3 d-none d-sm-block"></span>
+      <!-- This separator should be displayed only if at least one of the action menu or settings button are displayed  -->
+      <span
+        v-if="shouldShowActionsMenu || shouldShowSettingsButton"
+        aria-hidden="true"
+        class="gl-pl-3 border-left gl-mb-3 d-none d-sm-block"
+      ></span>
 
-        <div class="gl-mb-3 gl-mr-3 d-flex d-sm-block">
+      <div v-if="shouldShowActionsMenu" class="gl-mb-3 gl-mr-3 d-flex d-sm-block">
         <gl-new-dropdown
           v-gl-tooltip
           right
@@ -418,7 +429,16 @@ export default {
           </template>
         </gl-new-dropdown>
       </div>
-      </template>
+
+      <div v-if="shouldShowSettingsButton" class="mb-2 mr-2 d-flex d-sm-block">
+        <gl-button
+          v-gl-tooltip
+          data-testid="metrics-settings-button"
+          icon="settings"
+          :href="operationsSettingsPath"
+          :title="s__('Metrics|Metrics Settings')"
+        />
+      </div>
     </div>
     <duplicate-dashboard-modal
       :default-branch="defaultBranch"

app/assets/javascripts/monitoring/stores/state.js

@@ -64,8 +64,16 @@ export default () => ({
 
   // GitLab paths to other pages
   projectPath: null,
+  operationsSettingsPath: '',
   logsPath: invalidUrl,
 
   // static paths
   customDashboardBasePath: '',
+
+  // current user data
+  /**
+   * Flag that denotes if the currently logged user can access
+   * the project Settings -> Operations
+   */
+  canAccessOperationsSettings: false,
 });

app/assets/javascripts/monitoring/utils.js

@@ -25,6 +25,8 @@ export const stateAndPropsFromDataset = (dataset = {}) => {
     dashboardEndpoint,
     dashboardsEndpoint,
     dashboardTimezone,
+    canAccessOperationsSettings,
+    operationsSettingsPath,
     projectPath,
     logsPath,
     currentEnvironmentName,
@@ -44,6 +46,8 @@ export const stateAndPropsFromDataset = (dataset = {}) => {
       dashboardEndpoint,
       dashboardsEndpoint,
       dashboardTimezone,
+      canAccessOperationsSettings,
+      operationsSettingsPath,
       projectPath,
       logsPath,
       currentEnvironmentName,

app/helpers/environments_helper.rb

@@ -84,8 +84,9 @@ module EnvironmentsHelper
       'metrics-endpoint'     => additional_metrics_project_environment_path(project, environment, format: :json),
       'dashboard-endpoint'   => metrics_dashboard_project_environment_path(project, environment, format: :json),
       'deployments-endpoint' => project_environment_deployments_path(project, environment, format: :json),
-      'alerts-endpoint'      => project_prometheus_alerts_path(project, environment_id: environment.id, format: :json)
-
+      'alerts-endpoint'      => project_prometheus_alerts_path(project, environment_id: environment.id, format: :json),
+      'operations-settings-path' => project_settings_operations_path(project),
+      'can-access-operations-settings' => can?(current_user, :admin_operations, project).to_s
     }
   end
 

changelogs/unreleased/astoicescu-settingsButton.yml

+---
+title: Add a metrics settings button to the dashboard header
+merge_request: 35848
+author:
+type: added

ee/spec/helpers/ee/environments_helper_spec.rb

@@ -8,6 +8,10 @@ RSpec.describe EnvironmentsHelper do
   let(:user) { create(:user) }
 
   describe '#metrics_data' do
+    before do
+      allow(helper).to receive(:can?).and_return(false)
+    end
+
     subject { helper.metrics_data(project, environment) }
 
     context 'user has all accesses' do

locale/gitlab.pot

@@ -14577,6 +14577,9 @@ msgstr ""
 msgid "Metrics|Max"
 msgstr ""
 
+msgid "Metrics|Metrics Settings"
+msgstr ""
+
 msgid "Metrics|Min"
 msgstr ""
 

spec/frontend/monitoring/components/__snapshots__/dashboard_template_spec.js.snap

@@ -129,6 +129,10 @@ exports[`Dashboard template matches the default snapshot 1`] = `
       <!---->
        
       <!---->
+       
+      <!---->
+       
+      <!---->
     </div>
      
     <duplicate-dashboard-modal-stub

spec/frontend/monitoring/components/dashboard_header_spec.js

@@ -157,4 +157,52 @@ describe('Dashboard header', () => {
       });
     });
   });
+
+  describe('metrics settings button', () => {
+    const findSettingsButton = () => wrapper.find('[data-testid="metrics-settings-button"]');
+    const url = 'https://path/to/project/settings';
+
+    beforeEach(() => {
+      createShallowWrapper();
+
+      store.state.monitoringDashboard.canAccessOperationsSettings = false;
+      store.state.monitoringDashboard.operationsSettingsPath = '';
+    });
+
+    it('is rendered when the user can access the project settings and path to settings is available', () => {
+      store.state.monitoringDashboard.canAccessOperationsSettings = true;
+      store.state.monitoringDashboard.operationsSettingsPath = url;
+
+      return wrapper.vm.$nextTick(() => {
+        expect(findSettingsButton().exists()).toBe(true);
+      });
+    });
+
+    it('is not rendered when the user can not access the project settings', () => {
+      store.state.monitoringDashboard.canAccessOperationsSettings = false;
+      store.state.monitoringDashboard.operationsSettingsPath = url;
+
+      return wrapper.vm.$nextTick(() => {
+        expect(findSettingsButton().exists()).toBe(false);
+      });
+    });
+
+    it('is not rendered when the path to settings is unavailable', () => {
+      store.state.monitoringDashboard.canAccessOperationsSettings = false;
+      store.state.monitoringDashboard.operationsSettingsPath = '';
+
+      return wrapper.vm.$nextTick(() => {
+        expect(findSettingsButton().exists()).toBe(false);
+      });
+    });
+
+    it('leads to the project settings page', () => {
+      store.state.monitoringDashboard.canAccessOperationsSettings = true;
+      store.state.monitoringDashboard.operationsSettingsPath = url;
+
+      return wrapper.vm.$nextTick(() => {
+        expect(findSettingsButton().attributes('href')).toBe(url);
+      });
+    });
+  });
 });

spec/helpers/environments_helper_spec.rb

@@ -42,10 +42,26 @@ RSpec.describe EnvironmentsHelper do
         'custom-metrics-available' => 'true',
         'alerts-endpoint' => project_prometheus_alerts_path(project, environment_id: environment.id, format: :json),
         'prometheus-alerts-available' => 'true',
-        'custom-dashboard-base-path' => Metrics::Dashboard::CustomDashboardService::DASHBOARD_ROOT
+        'custom-dashboard-base-path' => Metrics::Dashboard::CustomDashboardService::DASHBOARD_ROOT,
+        'operations-settings-path' => project_settings_operations_path(project),
+        'can-access-operations-settings' => 'true'
       )
     end
 
+    context 'without admin_operations permission' do
+      before do
+        allow(helper).to receive(:can?)
+          .with(user, :admin_operations, project)
+          .and_return(false)
+      end
+
+      specify do
+        expect(metrics_data).to include(
+          'can-access-operations-settings' => 'false'
+        )
+      end
+    end
+
     context 'without read_prometheus_alerts permission' do
       before do
         allow(helper).to receive(:can?)