diff options
author | Grzegorz Bizon <grzesiek.bizon@gmail.com> | 2016-12-14 12:21:16 +0300 |
---|---|---|
committer | Grzegorz Bizon <grzesiek.bizon@gmail.com> | 2016-12-14 12:21:16 +0300 |
commit | ac115a9e1f106863112ca9daf463a8f5b8db7d0a (patch) | |
tree | b0699a35af173c106e60835684b4086ad4adafa5 /spec/models/generic_commit_status_spec.rb | |
parent | 309580bd41a178a02894d399e3cd45659a8ba7a7 (diff) |
Add some missing tests for detailed status methods
Diffstat (limited to 'spec/models/generic_commit_status_spec.rb')
-rw-r--r-- | spec/models/generic_commit_status_spec.rb | 16 |
1 files changed, 14 insertions, 2 deletions
diff --git a/spec/models/generic_commit_status_spec.rb b/spec/models/generic_commit_status_spec.rb index 615cfe3142b..6004bfdb7b7 100644 --- a/spec/models/generic_commit_status_spec.rb +++ b/spec/models/generic_commit_status_spec.rb @@ -1,8 +1,11 @@ require 'spec_helper' describe GenericCommitStatus, models: true do - let(:pipeline) { FactoryGirl.create :ci_pipeline } - let(:generic_commit_status) { FactoryGirl.create :generic_commit_status, pipeline: pipeline } + let(:pipeline) { create(:ci_pipeline) } + + let(:generic_commit_status) do + create(:generic_commit_status, pipeline: pipeline) + end describe '#context' do subject { generic_commit_status.context } @@ -17,6 +20,15 @@ describe GenericCommitStatus, models: true do it { is_expected.to eq([:external]) } end + describe '#detailed_status' do + let(:user) { create(:user) } + + it 'returns detailed status object' do + expect(generic_commit_status.detailed_status(user)) + .to be_a Gitlab::Ci::Status::Success + end + end + describe 'set_default_values' do before do generic_commit_status.context = nil |