diff options
author | Grzegorz Bizon <grzesiek.bizon@gmail.com> | 2017-03-06 14:43:32 +0100 |
---|---|---|
committer | Grzegorz Bizon <grzesiek.bizon@gmail.com> | 2017-03-06 14:43:32 +0100 |
commit | c4d5b231ed5888367912db000ebd8e2856709bc0 (patch) | |
tree | 87c8eb81df3bf9d132563c3c22af55a1c73e48f7 /spec/models/commit_status_spec.rb | |
parent | a004a9743a356597b97cc38103bd193cf6660753 (diff) | |
download | gitlab-ce-c4d5b231ed5888367912db000ebd8e2856709bc0.tar.gz |
Ignore optional actions when calculating warnings
Diffstat (limited to 'spec/models/commit_status_spec.rb')
-rw-r--r-- | spec/models/commit_status_spec.rb | 19 |
1 files changed, 19 insertions, 0 deletions
diff --git a/spec/models/commit_status_spec.rb b/spec/models/commit_status_spec.rb index da339b143a8..fee3c001e22 100644 --- a/spec/models/commit_status_spec.rb +++ b/spec/models/commit_status_spec.rb @@ -195,6 +195,25 @@ describe CommitStatus, :models do end end + describe '.failed_but_allowed' do + subject { described_class.failed_but_allowed.order(:id) } + + let(:statuses) do + [create_status(allow_failure: true, status: 'success'), + create_status(allow_failure: true, status: 'failed'), + create_status(allow_failure: false, status: 'success'), + create_status(allow_failure: false, status: 'failed'), + create_status(allow_failure: true, status: 'canceled'), + create_status(allow_failure: false, status: 'canceled'), + create_status(allow_failure: true, status: 'manual'), + create_status(allow_failure: false, status: 'manual')] + end + + it 'returns statuses without what we want to ignore' do + is_expected.to eq(statuses.values_at(1, 4)) + end + end + describe '#before_sha' do subject { commit_status.before_sha } |