Merge branch 'refactor-update-dashboard-service-tests' into 'master'

Refactor UpdateDashboardService tests

See merge request gitlab-org/gitlab!28376

spec/services/metrics/dashboard/update_dashboard_service_spec.rb

@@ -27,14 +27,7 @@ describe Metrics::Dashboard::UpdateDashboardService, :use_clean_rails_memory_sto
     end
 
     context 'user does not have push right to repository' do
-      it 'returns an appropriate message and status code', :aggregate_failures do
-        result = service_call
-
-        expect(result.keys).to contain_exactly(:message, :http_status, :status, :last_step)
-        expect(result[:status]).to eq(:error)
-        expect(result[:http_status]).to eq(:forbidden)
-        expect(result[:message]).to eq("You are not allowed to push into this branch. Create another branch or open a merge request.")
-      end
+      it_behaves_like 'misconfigured dashboard service response with stepable', :forbidden, 'You are not allowed to push into this branch. Create another branch or open a merge request.'
     end
 
     context 'with rights to push to the repository' do
@@ -46,27 +39,13 @@ describe Metrics::Dashboard::UpdateDashboardService, :use_clean_rails_memory_sto
         context 'with a yml extension' do
           let(:file_name) { 'config/prometheus/../database.yml' }
 
-          it 'returns an appropriate message and status code', :aggregate_failures do
-            result = service_call
-
-            expect(result.keys).to contain_exactly(:message, :http_status, :status, :last_step)
-            expect(result[:status]).to eq(:error)
-            expect(result[:http_status]).to eq(:bad_request)
-            expect(result[:message]).to eq("A file with this name doesn't exist")
-          end
+          it_behaves_like 'misconfigured dashboard service response with stepable', :bad_request, "A file with this name doesn't exist"
         end
 
         context 'without a yml extension' do
           let(:file_name) { '../../..../etc/passwd' }
 
-          it 'returns an appropriate message and status code', :aggregate_failures do
-            result = service_call
-
-            expect(result.keys).to contain_exactly(:message, :http_status, :status, :last_step)
-            expect(result[:status]).to eq(:error)
-            expect(result[:http_status]).to eq(:bad_request)
-            expect(result[:message]).to eq("The file name should have a .yml extension")
-          end
+          it_behaves_like 'misconfigured dashboard service response with stepable', :bad_request, 'The file name should have a .yml extension'
         end
       end
 
@@ -81,14 +60,7 @@ describe Metrics::Dashboard::UpdateDashboardService, :use_clean_rails_memory_sto
           project.repository.add_branch(user, branch, 'master')
         end
 
-        it 'returns an appropriate message and status code', :aggregate_failures do
-          result = service_call
-
-          expect(result.keys).to contain_exactly(:message, :http_status, :status, :last_step)
-          expect(result[:status]).to eq(:error)
-          expect(result[:http_status]).to eq(:bad_request)
-          expect(result[:message]).to eq("There was an error updating the dashboard, branch named: existing_branch already exists.")
-        end
+        it_behaves_like 'misconfigured dashboard service response with stepable', :bad_request, 'There was an error updating the dashboard, branch named: existing_branch already exists.'
       end
 
       context 'Files::UpdateService success' do