diff options
author | Kamil Trzcinski <ayufan@ayufan.eu> | 2016-08-11 23:54:25 +0300 |
---|---|---|
committer | Kamil Trzcinski <ayufan@ayufan.eu> | 2016-08-11 23:54:25 +0300 |
commit | 4ccf39cde7356bf98bef5aae694257fb2c001e75 (patch) | |
tree | e5acf2ae15a7223eeb45811ded8446dcd9d57c64 /spec/lib/ci/charts_spec.rb | |
parent | 6a6a69f4afbe0107a75df018b662f02b5ec0166a (diff) |
Fix test failures, that did occur because of missing previously used `reload_status!` call
Diffstat (limited to 'spec/lib/ci/charts_spec.rb')
-rw-r--r-- | spec/lib/ci/charts_spec.rb | 16 |
1 files changed, 9 insertions, 7 deletions
diff --git a/spec/lib/ci/charts_spec.rb b/spec/lib/ci/charts_spec.rb index 034ea098193..fb6cc398307 100644 --- a/spec/lib/ci/charts_spec.rb +++ b/spec/lib/ci/charts_spec.rb @@ -2,21 +2,23 @@ require 'spec_helper' describe Ci::Charts, lib: true do context "build_times" do + let(:project) { create(:empty_project) } + let(:chart) { Ci::Charts::BuildTime.new(project) } + + subject { chart.build_times } + before do - @pipeline = FactoryGirl.create(:ci_pipeline) - FactoryGirl.create(:ci_build, pipeline: @pipeline) + create(:ci_empty_pipeline, project: project, duration: 120) end it 'returns build times in minutes' do - chart = Ci::Charts::BuildTime.new(@pipeline.project) - expect(chart.build_times).to eq([2]) + is_expected.to contain_exactly(2) end it 'handles nil build times' do - create(:ci_pipeline, duration: nil, project: @pipeline.project) + create(:ci_empty_pipeline, project: project, duration: nil) - chart = Ci::Charts::BuildTime.new(@pipeline.project) - expect(chart.build_times).to eq([2, 0]) + is_expected.to contain_exactly(2, 0) end end end |