Extract context in JobsController spec

spec/controllers/projects/jobs_controller_spec.rb

@@ -398,15 +398,16 @@ describe Projects::JobsController, :clean_gitlab_redis_shared_state do
         end
       end
 
-      context 'with variables and user is a maintainer' do
+      context 'with variables' do
         before do
-          project.add_maintainer(user)
-
           create(:ci_pipeline_variable, pipeline: pipeline, key: :TRIGGER_KEY_1, value: 'TRIGGER_VALUE_1')
+        end
 
-          get_show(id: job.id, format: :json)
+        context 'user is a maintainer' do
+          before do
+            project.add_maintainer(user)
 
-          @first_variable = json_response['trigger']['variables'].first
+            get_show(id: job.id, format: :json)
           end
 
           it 'returns a job_detail' do
@@ -420,19 +421,17 @@ describe Projects::JobsController, :clean_gitlab_redis_shared_state do
           end
 
           it 'exposes correct variable properties' do
-          expect(@first_variable['key']).to eq "TRIGGER_KEY_1"
-          expect(@first_variable['value']).to eq "TRIGGER_VALUE_1"
-          expect(@first_variable['public']).to eq false
+            first_variable = json_response['trigger']['variables'].first
+
+            expect(first_variable['key']).to eq "TRIGGER_KEY_1"
+            expect(first_variable['value']).to eq "TRIGGER_VALUE_1"
+            expect(first_variable['public']).to eq false
           end
         end
 
-      context 'with variables and user is not a mantainer' do
+        context 'user is not a mantainer' do
           before do
-          create(:ci_pipeline_variable, pipeline: pipeline, key: :TRIGGER_KEY_1, value: 'TRIGGER_VALUE_1')
-
             get_show(id: job.id, format: :json)
-
-          @first_variable = json_response['trigger']['variables'].first
           end
 
           it 'returns a job_detail' do
@@ -446,9 +445,12 @@ describe Projects::JobsController, :clean_gitlab_redis_shared_state do
           end
 
           it 'exposes correct variable properties' do
-          expect(@first_variable['key']).to eq "TRIGGER_KEY_1"
-          expect(@first_variable['value']).to be_nil
-          expect(@first_variable['public']).to eq false
+            first_variable = json_response['trigger']['variables'].first
+
+            expect(first_variable['key']).to eq "TRIGGER_KEY_1"
+            expect(first_variable['value']).to be_nil
+            expect(first_variable['public']).to eq false
+          end
         end
       end
     end