diff options
Diffstat (limited to 'spec/migrations')
-rw-r--r-- | spec/migrations/20221102090940_create_next_ci_partitions_record_spec.rb | 63 | ||||
-rw-r--r-- | spec/migrations/20221102090943_create_second_partition_for_builds_metadata_spec.rb | 61 |
2 files changed, 124 insertions, 0 deletions
diff --git a/spec/migrations/20221102090940_create_next_ci_partitions_record_spec.rb b/spec/migrations/20221102090940_create_next_ci_partitions_record_spec.rb new file mode 100644 index 00000000000..c55e4bcfba7 --- /dev/null +++ b/spec/migrations/20221102090940_create_next_ci_partitions_record_spec.rb @@ -0,0 +1,63 @@ +# frozen_string_literal: true + +require 'spec_helper' +require_migration! + +RSpec.describe CreateNextCiPartitionsRecord, migration: :gitlab_ci do + let(:migration) { described_class.new } + let(:partitions) { table(:ci_partitions) } + + describe '#up' do + context 'when on sass' do + before do + allow(Gitlab).to receive(:com?).and_return(true) + end + + it 'creates next partitions record and resets the sequence' do + expect { migrate! } + .to change { partitions.where(id: 101).any? } + .from(false).to(true) + + expect { partitions.create! }.not_to raise_error + end + end + + context 'when self-managed' do + before do + allow(Gitlab).to receive(:com?).and_return(false) + end + + it 'does not create records' do + expect { migrate! }.not_to change(partitions, :count) + end + end + end + + describe '#down' do + context 'when on sass' do + before do + allow(Gitlab).to receive(:com?).and_return(true) + end + + it 'removes the record' do + migrate! + + expect { migration.down } + .to change { partitions.where(id: 101).any? } + .from(true).to(false) + end + end + + context 'when self-managed' do + before do + allow(Gitlab).to receive(:com?).and_return(true, false) + end + + it 'does not remove the record' do + expect { migrate! }.to change(partitions, :count).by(1) + + expect { migration.down }.not_to change(partitions, :count) + end + end + end +end diff --git a/spec/migrations/20221102090943_create_second_partition_for_builds_metadata_spec.rb b/spec/migrations/20221102090943_create_second_partition_for_builds_metadata_spec.rb new file mode 100644 index 00000000000..99754d609ed --- /dev/null +++ b/spec/migrations/20221102090943_create_second_partition_for_builds_metadata_spec.rb @@ -0,0 +1,61 @@ +# frozen_string_literal: true + +require 'spec_helper' +require_migration! + +RSpec.describe CreateSecondPartitionForBuildsMetadata, :migration do + let(:migration) { described_class.new } + let(:partitions) { table(:ci_partitions) } + + describe '#up' do + context 'when on sass' do + before do + allow(Gitlab).to receive(:com?).and_return(true) + end + + it 'creates a new partition' do + expect { migrate! }.to change { partitions_count }.by(1) + end + end + + context 'when self-managed' do + before do + allow(Gitlab).to receive(:com?).and_return(false) + end + + it 'does not create the partition' do + expect { migrate! }.not_to change { partitions_count } + end + end + end + + describe '#down' do + context 'when on sass' do + before do + allow(Gitlab).to receive(:com?).and_return(true) + end + + it 'removes the partition' do + migrate! + + expect { migration.down }.to change { partitions_count }.by(-1) + end + end + + context 'when self-managed' do + before do + allow(Gitlab).to receive(:com?).and_return(false) + end + + it 'does not change the partitions count' do + migrate! + + expect { migration.down }.not_to change { partitions_count } + end + end + end + + def partitions_count + Gitlab::Database::PostgresPartition.for_parent_table(:p_ci_builds_metadata).size + end +end |