Commit c0716421 authored by Matija Čupić's avatar Matija Čupić

Refactor pipeline id ordering

parent 0b7ff7c5
...@@ -6,6 +6,7 @@ describe Gitlab::BackgroundMigration::PopulateExternalPipelineSource, :migration ...@@ -6,6 +6,7 @@ describe Gitlab::BackgroundMigration::PopulateExternalPipelineSource, :migration
let(:migration) { described_class.new } let(:migration) { described_class.new }
let!(:internal_pipeline) { create(:ci_pipeline, source: :web) } let!(:internal_pipeline) { create(:ci_pipeline, source: :web) }
let(:pipelines) { [internal_pipeline, external_pipeline, second_external_pipeline].map(&:id) }
let!(:external_pipeline) do let!(:external_pipeline) do
build(:ci_pipeline, source: :unknown) build(:ci_pipeline, source: :unknown)
...@@ -21,7 +22,7 @@ describe Gitlab::BackgroundMigration::PopulateExternalPipelineSource, :migration ...@@ -21,7 +22,7 @@ describe Gitlab::BackgroundMigration::PopulateExternalPipelineSource, :migration
create(:ci_build, pipeline: internal_pipeline) create(:ci_build, pipeline: internal_pipeline)
end end
subject { migration.perform(external_pipeline.id, second_external_pipeline.id) } subject { migration.perform(pipelines.min, pipelines.max) }
it 'populates the pipeline source' do it 'populates the pipeline source' do
subject subject
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment