2018-01-19 13:25:30 +00:00
|
|
|
require 'spec_helper'
|
|
|
|
require Rails.root.join('db', 'post_migrate', '20180119121225_remove_redundant_pipeline_stages.rb')
|
|
|
|
|
|
|
|
describe RemoveRedundantPipelineStages, :migration do
|
|
|
|
let(:projects) { table(:projects) }
|
|
|
|
let(:pipelines) { table(:ci_pipelines) }
|
|
|
|
let(:stages) { table(:ci_stages) }
|
|
|
|
let(:builds) { table(:ci_builds) }
|
|
|
|
|
|
|
|
before do
|
|
|
|
projects.create!(id: 123, name: 'gitlab', path: 'gitlab-ce')
|
|
|
|
pipelines.create!(id: 234, project_id: 123, ref: 'master', sha: 'adf43c3a')
|
|
|
|
|
|
|
|
stages.create!(id: 6, project_id: 123, pipeline_id: 234, name: 'build')
|
|
|
|
stages.create!(id: 10, project_id: 123, pipeline_id: 234, name: 'build')
|
|
|
|
stages.create!(id: 21, project_id: 123, pipeline_id: 234, name: 'build')
|
|
|
|
stages.create!(id: 41, project_id: 123, pipeline_id: 234, name: 'test')
|
|
|
|
stages.create!(id: 62, project_id: 123, pipeline_id: 234, name: 'test')
|
|
|
|
stages.create!(id: 102, project_id: 123, pipeline_id: 234, name: 'deploy')
|
|
|
|
|
|
|
|
builds.create!(id: 1, commit_id: 234, project_id: 123, stage_id: 10)
|
|
|
|
builds.create!(id: 2, commit_id: 234, project_id: 123, stage_id: 21)
|
|
|
|
builds.create!(id: 3, commit_id: 234, project_id: 123, stage_id: 21)
|
|
|
|
builds.create!(id: 4, commit_id: 234, project_id: 123, stage_id: 41)
|
|
|
|
builds.create!(id: 5, commit_id: 234, project_id: 123, stage_id: 62)
|
|
|
|
builds.create!(id: 6, commit_id: 234, project_id: 123, stage_id: 102)
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'removes ambiguous stages and preserves builds' do
|
|
|
|
expect(stages.all.count).to eq 6
|
|
|
|
expect(builds.all.count).to eq 6
|
|
|
|
|
|
|
|
migrate!
|
|
|
|
|
|
|
|
expect(stages.all.count).to eq 1
|
|
|
|
expect(builds.all.count).to eq 6
|
|
|
|
expect(builds.all.pluck(:stage_id).compact).to eq [102]
|
|
|
|
end
|
2018-02-05 15:22:19 +00:00
|
|
|
|
|
|
|
it 'retries when duplicated stages are being created during migration' do
|
|
|
|
allow(subject).to receive(:remove_outdated_index!)
|
|
|
|
expect(subject).to receive(:remove_redundant_pipeline_stages!).exactly(3).times
|
|
|
|
allow(subject).to receive(:add_unique_index!)
|
|
|
|
.and_raise(ActiveRecord::RecordNotUnique.new('Duplicated stages present!'))
|
|
|
|
|
|
|
|
expect { subject.up(attempts: 3) }.to raise_error ActiveRecord::RecordNotUnique
|
|
|
|
end
|
2018-01-19 13:25:30 +00:00
|
|
|
end
|