Commit c2b3559b authored by Grzegorz Bizon's avatar Grzegorz Bizon

Verify new pipeline failure reason in the service specs

parent 17a0a6cc
...@@ -40,11 +40,11 @@ describe Ci::CreatePipelineService, '#execute' do ...@@ -40,11 +40,11 @@ describe Ci::CreatePipelineService, '#execute' do
it 'drops the pipeline and cancels all jobs' do it 'drops the pipeline and cancels all jobs' do
pipeline = create_pipeline! pipeline = create_pipeline!
# TODO, check failure reason
expect(pipeline).to be_persisted expect(pipeline).to be_persisted
expect(pipeline).to be_failed expect(pipeline).to be_failed
expect(pipeline.statuses).not_to be_empty expect(pipeline.statuses).not_to be_empty
expect(pipeline.statuses).to all(be_canceled) expect(pipeline.statuses).to all(be_canceled)
expect(pipeline.activity_limit_exceeded?).to be true
end end
end end
...@@ -56,11 +56,11 @@ describe Ci::CreatePipelineService, '#execute' do ...@@ -56,11 +56,11 @@ describe Ci::CreatePipelineService, '#execute' do
it 'drops pipeline without creating jobs' do it 'drops pipeline without creating jobs' do
pipeline = create_pipeline! pipeline = create_pipeline!
# TODO, check failure reason
expect(pipeline).to be_persisted expect(pipeline).to be_persisted
expect(pipeline).to be_failed expect(pipeline).to be_failed
expect(pipeline.seeds_size).to be > 2 expect(pipeline.seeds_size).to be > 2
expect(pipeline.statuses).to be_empty expect(pipeline.statuses).to be_empty
expect(pipeline.size_limit_exceeded?).to be true
end end
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