diff options
author | Robert Speicher <robert@gitlab.com> | 2016-09-10 01:26:32 +0000 |
---|---|---|
committer | Robert Speicher <robert@gitlab.com> | 2016-09-10 01:26:32 +0000 |
commit | d01cbe0c8baf0ab630cf5e5d28087e91c8679b70 (patch) | |
tree | 1e6844877db368e98880c1aa9f3968b597225b9b | |
parent | 1417437590a347e30b1a3ee96b218f8eaf635366 (diff) | |
parent | e300e9d5ef032aa6543f471dcea37ba068c98a5e (diff) | |
download | gitlab-ce-d01cbe0c8baf0ab630cf5e5d28087e91c8679b70.tar.gz |
Merge branch 'pipelines-finder-spec-fix' into 'master'
Improve PipelinesFinder spec so that it does not depend on hard-coded database IDs
Fixes failed builds such as https://gitlab.com/gitlab-org/gitlab-ce/builds/3919501
See merge request !6291
-rw-r--r-- | spec/finders/pipelines_finder_spec.rb | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/spec/finders/pipelines_finder_spec.rb b/spec/finders/pipelines_finder_spec.rb index 7100266ab55..b0811d134fa 100644 --- a/spec/finders/pipelines_finder_spec.rb +++ b/spec/finders/pipelines_finder_spec.rb @@ -43,9 +43,10 @@ describe PipelinesFinder do let(:params) { { scope: 'running' } } it 'orders in descending order on ID' do - create(:ci_pipeline, project: project, ref: 'feature') + feature_pipeline = create(:ci_pipeline, project: project, ref: 'feature') - expect(subject.map(&:id)).to eq [3, 2, 1] + expected_ids = [feature_pipeline.id, branch_pipeline.id, tag_pipeline.id].sort.reverse + expect(subject.map(&:id)).to eq expected_ids end end end |