Merge branch 'add_missing_spec_context_to_pipeline_data_builder' into 'master'

Add 'with runner' context to pipeline data_builder

See merge request gitlab-org/gitlab!48393

spec/lib/gitlab/data_builder/pipeline_spec.rb

@@ -21,9 +21,10 @@ RSpec.describe Gitlab::DataBuilder::Pipeline do
     let(:data) { described_class.build(pipeline) }
     let(:attributes) { data[:object_attributes] }
     let(:build_data) { data[:builds].first }
+    let(:runner_data) { build_data[:runner] }
     let(:project_data) { data[:project] }
 
-    it 'has correct attributes' do
+    it 'has correct attributes', :aggregate_failures do
       expect(attributes).to be_a(Hash)
       expect(attributes[:ref]).to eq(pipeline.ref)
       expect(attributes[:sha]).to eq(pipeline.sha)
@@ -36,6 +37,7 @@ RSpec.describe Gitlab::DataBuilder::Pipeline do
       expect(build_data[:id]).to eq(build.id)
       expect(build_data[:status]).to eq(build.status)
       expect(build_data[:allow_failure]).to eq(build.allow_failure)
+      expect(runner_data).to eq(nil)
       expect(project_data).to eq(project.hook_attrs(backward: false))
       expect(data[:merge_request]).to be_nil
       expect(data[:user]).to eq({
@@ -46,6 +48,18 @@ RSpec.describe Gitlab::DataBuilder::Pipeline do
         })
     end
 
+    context 'build with runner' do
+      let!(:build) { create(:ci_build, pipeline: pipeline, runner: ci_runner) }
+      let(:ci_runner) { create(:ci_runner) }
+
+      it 'has runner attributes', :aggregate_failures do
+        expect(runner_data[:id]).to eq(ci_runner.id)
+        expect(runner_data[:description]).to eq(ci_runner.description)
+        expect(runner_data[:active]).to eq(ci_runner.active)
+        expect(runner_data[:is_shared]).to eq(ci_runner.instance_type?)
+      end
+    end
+
     context 'pipeline without variables' do
       it 'has empty variables hash' do
         expect(attributes[:variables]).to be_a(Array)