diff options
author | mfluharty <mfluharty@gitlab.com> | 2019-04-19 22:19:59 +0300 |
---|---|---|
committer | mfluharty <mfluharty@gitlab.com> | 2019-04-19 23:03:53 +0300 |
commit | 6b7b07b7ec0bc434716236d18d34c9a4a4d2f2f4 (patch) | |
tree | 65275f0892f54e4a86f9b9c82347f2431bfa7c87 /spec/lib/gitlab/ci/status | |
parent | 2c2bd00a6c2f4484ac7ba522b8d6b5fb64304a42 (diff) |
Update specs that use retryable canceled jobs
Specs that test canceled jobs now
- expect them not to be retryable or playable
- expect them not to show retry buttons
Specs that test retryability now
- use failed status instead of canceled status
Diffstat (limited to 'spec/lib/gitlab/ci/status')
-rw-r--r-- | spec/lib/gitlab/ci/status/build/factory_spec.rb | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/spec/lib/gitlab/ci/status/build/factory_spec.rb b/spec/lib/gitlab/ci/status/build/factory_spec.rb index b6231510b91..025439f1b6e 100644 --- a/spec/lib/gitlab/ci/status/build/factory_spec.rb +++ b/spec/lib/gitlab/ci/status/build/factory_spec.rb @@ -163,11 +163,11 @@ describe Gitlab::Ci::Status::Build::Factory do it 'matches correct extended statuses' do expect(factory.extended_statuses) - .to eq [Gitlab::Ci::Status::Build::Canceled, Gitlab::Ci::Status::Build::Retryable] + .to eq [Gitlab::Ci::Status::Build::Canceled] end - it 'fabricates a retryable build status' do - expect(status).to be_a Gitlab::Ci::Status::Build::Retryable + it 'does not fabricate a retryable build status' do + expect(status).not_to be_a Gitlab::Ci::Status::Build::Retryable end it 'fabricates status with correct details' do @@ -177,7 +177,7 @@ describe Gitlab::Ci::Status::Build::Factory do expect(status.illustration).to include(:image, :size, :title) expect(status.label).to eq 'canceled' expect(status).to have_details - expect(status).to have_action + expect(status).not_to have_action end end |