diff options
author | Kamil Trzciński <ayufan@ayufan.eu> | 2018-07-30 17:05:50 +0200 |
---|---|---|
committer | Kamil Trzciński <ayufan@ayufan.eu> | 2018-08-01 12:25:19 +0200 |
commit | 8ce4bf20e737482db33f9ad1d37ad8212c6330a0 (patch) | |
tree | 4d04ade2163f56baf4fccfe985692806828e317f /spec | |
parent | 8ce03988b9a29d34016052be2dd8641fa7b9efbe (diff) | |
download | gitlab-ce-8ce4bf20e737482db33f9ad1d37ad8212c6330a0.tar.gz |
Improve CommitStatusPresenter
Diffstat (limited to 'spec')
-rw-r--r-- | spec/presenters/commit_status_presenter_spec.rb | 26 |
1 files changed, 26 insertions, 0 deletions
diff --git a/spec/presenters/commit_status_presenter_spec.rb b/spec/presenters/commit_status_presenter_spec.rb index f81ee44e371..2b7742ddbb8 100644 --- a/spec/presenters/commit_status_presenter_spec.rb +++ b/spec/presenters/commit_status_presenter_spec.rb @@ -12,4 +12,30 @@ describe CommitStatusPresenter do it 'inherits from Gitlab::View::Presenter::Delegated' do expect(described_class.superclass).to eq(Gitlab::View::Presenter::Delegated) end + + describe 'covers all failure reasons' do + let(:message) { presenter.callout_failure_message } + + CommitStatus.failure_reasons.keys.each do |failure_reason| + context failure_reason do + before do + build.failure_reason = failure_reason + end + + it "is a valid status" do + expect { message }.not_to raise_error + end + end + end + + context 'invalid failure message' do + before do + expect(build).to receive(:failure_reason) { 'invalid failure message' } + end + + it "is an invalid status" do + expect { message }.to raise_error(/key not found:/) + end + end + end end |