Commit 51d6fab3 authored by drew cimino's avatar drew cimino

Use some doubles and let_it_be

parent eb36de62
...@@ -3,11 +3,23 @@ ...@@ -3,11 +3,23 @@
require 'spec_helper' require 'spec_helper'
RSpec.describe Gitlab::Badge::Coverage::Report do RSpec.describe Gitlab::Badge::Coverage::Report do
let(:project) { create(:project, :repository) } let(:project) { double('project') }
let(:job_name) { nil }
let_it_be(:successful_pipeline) do
create(:ci_pipeline, :success)
end
let(:badge) do let(:badge) do
described_class.new(project, 'master', opts: { job: job_name }) described_class.new(project, 'master', opts: { job: job_name }).tap do |new_badge|
allow(new_badge).to receive(:pipeline).and_return(successful_pipeline)
end
end
let(:job_name) { nil }
let(:builds) { [] }
before do
allow(successful_pipeline).to receive(:builds).and_return(Ci::Build.where(id: builds.map(&:id)))
end end
describe '#entity' do describe '#entity' do
...@@ -47,15 +59,11 @@ RSpec.describe Gitlab::Badge::Coverage::Report do ...@@ -47,15 +59,11 @@ RSpec.describe Gitlab::Badge::Coverage::Report do
end end
context 'when latest successful pipeline exists' do context 'when latest successful pipeline exists' do
before do let(:builds) do
create_pipeline do |pipeline| [
create(:ci_build, :success, pipeline: pipeline, name: 'first', coverage: 40) create(:ci_build, :success, pipeline: pipeline, name: 'first', coverage: 40),
create(:ci_build, :success, pipeline: pipeline, coverage: 60) create(:ci_build, :success, pipeline: pipeline, coverage: 60)
end ]
create_pipeline do |pipeline|
create(:ci_build, :failed, pipeline: pipeline, coverage: 10)
end
end end
context 'when particular job specified' do context 'when particular job specified' do
...@@ -78,13 +86,4 @@ RSpec.describe Gitlab::Badge::Coverage::Report do ...@@ -78,13 +86,4 @@ RSpec.describe Gitlab::Badge::Coverage::Report do
context 'pipeline does not exist' do context 'pipeline does not exist' do
it_behaves_like 'unknown coverage report' it_behaves_like 'unknown coverage report'
end end
def create_pipeline
opts = { project: project, sha: project.commit.id, ref: 'master' }
create(:ci_pipeline, opts).tap do |pipeline|
yield pipeline
::Ci::ProcessPipelineService.new(pipeline).execute
end
end
end end
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment