If user can't activate error tracking display a learn more button pointing

  to doc

2 files changed, 52 insertions, 8 deletions

diff --git a/spec/frontend/error_tracking/components/error_tracking_list_spec.js b/spec/frontend/error_tracking/components/error_tracking_list_spec.js
index 67e5dc399ac..ce8b8908026 100644
--- a/spec/frontend/error_tracking/components/error_tracking_list_spec.js
+++ b/spec/frontend/error_tracking/components/error_tracking_list_spec.js
@@ -11,19 +11,24 @@ describe('ErrorTrackingList', () => {
   let wrapper;
   let actions;
 
-  function mountComponent({ errorTrackingEnabled = true } = {}) {
+  function mountComponent({
+    errorTrackingEnabled = true,
+    userCanEnableErrorTracking = true,
+    stubs = {
+      'gl-link': GlLink,
+    },
+  } = {}) {
     wrapper = shallowMount(ErrorTrackingList, {
       localVue,
       store,
       propsData: {
         indexPath: '/path',
         enableErrorTrackingLink: '/link',
+        userCanEnableErrorTracking,
         errorTrackingEnabled,
         illustrationPath: 'illustration/path',
       },
-      stubs: {
-        'gl-link': GlLink,
-      },
+      stubs,
     });
   }
 
@@ -115,4 +120,23 @@ describe('ErrorTrackingList', () => {
       expect(wrapper.find(GlButton).exists()).toBeFalsy();
     });
   });
+
+  describe('When error tracking is disabled and user is not allowed to enable it', () => {
+    beforeEach(() => {
+      mountComponent({
+        errorTrackingEnabled: false,
+        userCanEnableErrorTracking: false,
+        stubs: {
+          'gl-link': GlLink,
+          'gl-empty-state': GlEmptyState,
+        },
+      });
+    });
+
+    it('shows empty state', () => {
+      expect(wrapper.find('a').attributes('href')).toBe(
+        '/help/user/project/operations/error_tracking.html',
+      );
+    });
+  });
 });
diff --git a/spec/helpers/projects/error_tracking_helper_spec.rb b/spec/helpers/projects/error_tracking_helper_spec.rb
index 7516a636c93..064b3ad21cb 100644
--- a/spec/helpers/projects/error_tracking_helper_spec.rb
+++ b/spec/helpers/projects/error_tracking_helper_spec.rb
@@ -6,21 +6,31 @@ describe Projects::ErrorTrackingHelper do
   include Gitlab::Routing.url_helpers
 
   set(:project) { create(:project) }
+  set(:current_user) { create(:user) }
 
   describe '#error_tracking_data' do
+    let(:can_enable_error_tracking) { true }
     let(:setting_path) { project_settings_operations_path(project) }
 
     let(:index_path) do
       project_error_tracking_index_path(project, format: :json)
     end
 
+    before do
+      allow(helper)
+        .to receive(:can?)
+        .with(current_user, :admin_operations, project)
+        .and_return(can_enable_error_tracking)
+    end
+
     context 'without error_tracking_setting' do
       it 'returns frontend configuration' do
-        expect(error_tracking_data(project)).to eq(
+        expect(helper.error_tracking_data(current_user, project)).to match(
           'index-path' => index_path,
+          'user-can-enable-error-tracking' => 'true',
           'enable-error-tracking-link' => setting_path,
           'error-tracking-enabled' => 'false',
-          "illustration-path" => "/images/illustrations/cluster_popover.svg"
+          'illustration-path' => match_asset_path('/assets/illustrations/cluster_popover.svg')
         )
       end
     end
@@ -36,7 +46,7 @@ describe Projects::ErrorTrackingHelper do
         end
 
         it 'show error tracking enabled' do
-          expect(error_tracking_data(project)).to include(
+          expect(helper.error_tracking_data(current_user, project)).to include(
             'error-tracking-enabled' => 'true'
           )
         end
@@ -48,11 +58,21 @@ describe Projects::ErrorTrackingHelper do
         end
 
         it 'show error tracking not enabled' do
-          expect(error_tracking_data(project)).to include(
+          expect(helper.error_tracking_data(current_user, project)).to include(
             'error-tracking-enabled' => 'false'
           )
         end
       end
     end
+
+    context 'when user is not maintainer' do
+      let(:can_enable_error_tracking) { false }
+
+      it 'shows error tracking enablement as disabled' do
+        expect(helper.error_tracking_data(current_user, project)).to include(
+          'user-can-enable-error-tracking' => 'false'
+        )
+      end
+    end
   end
 end