diff options
author | Shinya Maeda <shinya@gitlab.com> | 2019-06-26 19:29:35 +0700 |
---|---|---|
committer | Shinya Maeda <shinya@gitlab.com> | 2019-07-04 09:53:40 +0700 |
commit | 93be669e16b71a00362187a690e573209d80d960 (patch) | |
tree | 6fc866212596ae5a2fbde9b917ea944363b7f8fa /spec | |
parent | 19dc1105524e3d25821670706a750043775588fa (diff) | |
download | gitlab-ce-93be669e16b71a00362187a690e573209d80d960.tar.gz |
Refactor pipeline errors_messagerefactor-pipeline-errors_message-ce
Use the shared method in Ci::Pipeline
Diffstat (limited to 'spec')
-rw-r--r-- | spec/models/ci/pipeline_spec.rb | 24 |
1 files changed, 24 insertions, 0 deletions
diff --git a/spec/models/ci/pipeline_spec.rb b/spec/models/ci/pipeline_spec.rb index 55cea48b641..e24bbc39761 100644 --- a/spec/models/ci/pipeline_spec.rb +++ b/spec/models/ci/pipeline_spec.rb @@ -2998,4 +2998,28 @@ describe Ci::Pipeline, :mailer do end end end + + describe '#error_messages' do + subject { pipeline.error_messages } + + before do + pipeline.valid? + end + + context 'when pipeline has errors' do + let(:pipeline) { build(:ci_pipeline, sha: nil, ref: nil) } + + it 'returns the full error messages' do + is_expected.to eq("Sha can't be blank and Ref can't be blank") + end + end + + context 'when pipeline does not have errors' do + let(:pipeline) { build(:ci_pipeline) } + + it 'returns empty string' do + is_expected.to be_empty + end + end + end end |