diff options
author | Z.J. van de Weg <git@zjvandeweg.nl> | 2017-06-13 09:22:22 +0200 |
---|---|---|
committer | Z.J. van de Weg <git@zjvandeweg.nl> | 2017-06-13 11:19:45 +0200 |
commit | 93b555af85ff124820366855ed9b3a9fa719c272 (patch) | |
tree | ed23bb4b6c87656373c0bdab3941fe8a4bb935ad /spec | |
parent | de20057ccbd3b8c94d64ff5d8deb14cab232d08a (diff) | |
download | gitlab-ce-93b555af85ff124820366855ed9b3a9fa719c272.tar.gz |
Handle legacy jobs without namezj-commit-status-sortable-name
Older pipelines can contain jobs without a name, in which case
'nameless' is used right now, so we can properly still handle other
actions on these jobs.
Fixes gitlab-org/gitlab-ce#33403
Diffstat (limited to 'spec')
-rw-r--r-- | spec/models/ci/legacy_stage_spec.rb | 11 |
1 files changed, 11 insertions, 0 deletions
diff --git a/spec/models/ci/legacy_stage_spec.rb b/spec/models/ci/legacy_stage_spec.rb index 48116c7e701..d43c33d3807 100644 --- a/spec/models/ci/legacy_stage_spec.rb +++ b/spec/models/ci/legacy_stage_spec.rb @@ -55,6 +55,17 @@ describe Ci::LegacyStage, :models do expect(stage.groups.map(&:name)) .to eq %w[aaaaa rspec spinach] end + + context 'when a name is nil on legacy pipelines' do + before do + pipeline.builds.first.update_attribute(:name, nil) + end + + it 'returns an array of three groups' do + expect(stage.groups.map(&:name)) + .to eq ['', 'aaaaa', 'rspec', 'spinach'] + end + end end describe '#statuses_count' do |