From c4d5b231ed5888367912db000ebd8e2856709bc0 Mon Sep 17 00:00:00 2001 From: Grzegorz Bizon Date: Mon, 6 Mar 2017 14:43:32 +0100 Subject: Ignore optional actions when calculating warnings --- spec/models/commit_status_spec.rb | 19 +++++++++++++++++++ 1 file changed, 19 insertions(+) (limited to 'spec/models/commit_status_spec.rb') 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 } -- cgit v1.2.1