From 858d175c1527d650ea5d83e201777d0cf8ae84c9 Mon Sep 17 00:00:00 2001 From: GitLab Bot Date: Thu, 9 Mar 2023 17:30:09 +0000 Subject: Add latest changes from gitlab-org/gitlab@15-9-stable-ee --- ...0_finalize_backfill_user_details_fields_spec.rb | 109 +++++++++++++++++++++ 1 file changed, 109 insertions(+) create mode 100644 spec/migrations/20221102231130_finalize_backfill_user_details_fields_spec.rb (limited to 'spec/migrations') diff --git a/spec/migrations/20221102231130_finalize_backfill_user_details_fields_spec.rb b/spec/migrations/20221102231130_finalize_backfill_user_details_fields_spec.rb new file mode 100644 index 00000000000..37bff128edd --- /dev/null +++ b/spec/migrations/20221102231130_finalize_backfill_user_details_fields_spec.rb @@ -0,0 +1,109 @@ +# frozen_string_literal: true + +require 'spec_helper' + +require_migration! + +RSpec.describe FinalizeBackfillUserDetailsFields, :migration, feature_category: :user_management do + let(:batched_migrations) { table(:batched_background_migrations) } + let(:batch_failed_status) { 2 } + let(:batch_finalized_status) { 3 } + + let!(:migration) { described_class::BACKFILL_MIGRATION } + + describe '#up' do + shared_examples 'finalizes the migration' do + it 'finalizes the migration' do + expect do + migrate! + + migration_record.reload + failed_job.reload + end.to change { migration_record.status }.from(migration_record.status).to(3).and( + change { failed_job.status }.from(batch_failed_status).to(batch_finalized_status) + ) + end + end + + context 'when migration is missing' do + it 'warns migration not found' do + expect(Gitlab::AppLogger) + .to receive(:warn).with(/Could not find batched background migration for the given configuration:/) + + migrate! + end + end + + context 'with migration present' do + let!(:migration_record) do + batched_migrations.create!( + job_class_name: migration, + table_name: :users, + column_name: :id, + job_arguments: [], + interval: 2.minutes, + min_value: 1, + max_value: 2, + batch_size: 1000, + sub_batch_size: 500, + max_batch_size: 5000, + gitlab_schema: :gitlab_main, + status: 3 # finished + ) + end + + context 'when migration finished successfully' do + it 'does not raise exception' do + expect { migrate! }.not_to raise_error + end + end + + context 'when users.linkedin column has already been dropped' do + before do + table(:users).create!(id: 1, email: 'author@example.com', username: 'author', projects_limit: 10) + ActiveRecord::Base.connection.execute("ALTER TABLE users DROP COLUMN linkedin") + migration_record.update_column(:status, 1) + end + + after do + ActiveRecord::Base.connection.execute("ALTER TABLE users ADD COLUMN linkedin text DEFAULT '' NOT NULL") + end + + it 'does not raise exception' do + expect { migrate! }.not_to raise_error + end + end + + context 'with different migration statuses', :redis do + using RSpec::Parameterized::TableSyntax + + where(:status, :description) do + 0 | 'paused' + 1 | 'active' + 4 | 'failed' + 5 | 'finalizing' + end + + with_them do + let!(:failed_job) do + table(:batched_background_migration_jobs).create!( + batched_background_migration_id: migration_record.id, + status: batch_failed_status, + min_value: 1, + max_value: 10, + attempts: 2, + batch_size: 100, + sub_batch_size: 10 + ) + end + + before do + migration_record.update!(status: status) + end + + it_behaves_like 'finalizes the migration' + end + end + end + end +end -- cgit v1.2.3