Adjust specs for core pipeline detailed statuses
This commit is contained in:
parent
7319460c18
commit
c83be391ee
2 changed files with 25 additions and 3 deletions
|
@ -11,7 +11,8 @@ describe Gitlab::Ci::Status::Pipeline::Factory do
|
||||||
end
|
end
|
||||||
|
|
||||||
context 'when pipeline has a core status' do
|
context 'when pipeline has a core status' do
|
||||||
HasStatus::AVAILABLE_STATUSES.each do |simple_status|
|
(HasStatus::AVAILABLE_STATUSES - [HasStatus::BLOCKED_STATUS])
|
||||||
|
.each do |simple_status|
|
||||||
context "when core status is #{simple_status}" do
|
context "when core status is #{simple_status}" do
|
||||||
let(:pipeline) { create(:ci_pipeline, status: simple_status) }
|
let(:pipeline) { create(:ci_pipeline, status: simple_status) }
|
||||||
|
|
||||||
|
@ -23,7 +24,7 @@ describe Gitlab::Ci::Status::Pipeline::Factory do
|
||||||
expect(factory.core_status).to be_a expected_status
|
expect(factory.core_status).to be_a expected_status
|
||||||
end
|
end
|
||||||
|
|
||||||
it 'does not matche extended statuses' do
|
it 'does not match extended statuses' do
|
||||||
expect(factory.extended_statuses).to be_empty
|
expect(factory.extended_statuses).to be_empty
|
||||||
end
|
end
|
||||||
|
|
||||||
|
@ -39,6 +40,27 @@ describe Gitlab::Ci::Status::Pipeline::Factory do
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
|
|
||||||
|
context "when core status is manual" do
|
||||||
|
let(:pipeline) { create(:ci_pipeline, status: :manual) }
|
||||||
|
|
||||||
|
it "matches manual core status" do
|
||||||
|
expect(factory.core_status)
|
||||||
|
.to be_a Gitlab::Ci::Status::Manual
|
||||||
|
end
|
||||||
|
|
||||||
|
it 'matches a correct extended statuses' do
|
||||||
|
expect(factory.extended_statuses)
|
||||||
|
.to eq [Gitlab::Ci::Status::Pipeline::Blocked]
|
||||||
|
end
|
||||||
|
|
||||||
|
it 'extends core status with common pipeline methods' do
|
||||||
|
expect(status).to have_details
|
||||||
|
expect(status).not_to have_action
|
||||||
|
expect(status.details_path)
|
||||||
|
.to include "pipelines/#{pipeline.id}"
|
||||||
|
end
|
||||||
|
end
|
||||||
end
|
end
|
||||||
|
|
||||||
context 'when pipeline has warnings' do
|
context 'when pipeline has warnings' do
|
||||||
|
|
|
@ -647,7 +647,7 @@ describe Ci::Pipeline, models: true do
|
||||||
let(:pipeline) { create(:ci_pipeline, status: :manual) }
|
let(:pipeline) { create(:ci_pipeline, status: :manual) }
|
||||||
|
|
||||||
it 'returns detailed status for blocked pipeline' do
|
it 'returns detailed status for blocked pipeline' do
|
||||||
expect(subject.text).to eq 'manual'
|
expect(subject.text).to eq 'blocked'
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
|
|
||||||
|
|
Loading…
Reference in a new issue