1 files changed, 76 insertions, 0 deletions
diff --git a/spec/migrations/swap_merge_request_metrics_id_to_bigint_for_gitlab_dot_com_spec.rb b/spec/migrations/swap_merge_request_metrics_id_to_bigint_for_gitlab_dot_com_spec.rb
new file mode 100644
index 00000000000..b819495aa89
--- /dev/null
+++ b/spec/migrations/swap_merge_request_metrics_id_to_bigint_for_gitlab_dot_com_spec.rb
@@ -0,0 +1,76 @@
+# frozen_string_literal: true
+
+require 'spec_helper'
+require_migration!
+
+RSpec.describe SwapMergeRequestMetricsIdToBigintForGitlabDotCom, feature_category: :database do
+  describe '#up' do
+    before do
+      # As we call `schema_migrate_down!` before each example, and for this migration
+      # `#down` is same as `#up`, we need to ensure we start from the expected state.
+      connection = described_class.new.connection
+      connection.execute('ALTER TABLE merge_request_metrics ALTER COLUMN id TYPE integer')
+      connection.execute('ALTER TABLE merge_request_metrics ALTER COLUMN id_convert_to_bigint TYPE bigint')
+    end
+
+    it 'swaps the integer and bigint columns for GitLab.com, dev, or test' do
+      # rubocop: disable RSpec/AnyInstanceOf
+      allow_any_instance_of(described_class).to receive(:com_or_dev_or_test_but_not_jh?).and_return(true)
+      # rubocop: enable RSpec/AnyInstanceOf
+
+      merge_request_metrics = table(:merge_request_metrics)
+
+      disable_migrations_output do
+        reversible_migration do |migration|
+          migration.before -> {
+            merge_request_metrics.reset_column_information
+
+            expect(merge_request_metrics.columns.find { |c| c.name == 'id' }.sql_type).to eq('integer')
+            expect(merge_request_metrics.columns.find do |c|
+                     c.name == 'id_convert_to_bigint'
+                   end.sql_type).to eq('bigint')
+          }
+
+          migration.after -> {
+            merge_request_metrics.reset_column_information
+
+            expect(merge_request_metrics.columns.find { |c| c.name == 'id' }.sql_type).to eq('bigint')
+            expect(merge_request_metrics.columns.find do |c|
+                     c.name == 'id_convert_to_bigint'
+                   end.sql_type).to eq('integer')
+          }
+        end
+      end
+    end
+
+    it 'is a no-op for other instances' do
+      # rubocop: disable RSpec/AnyInstanceOf
+      allow_any_instance_of(described_class).to receive(:com_or_dev_or_test_but_not_jh?).and_return(false)
+      # rubocop: enable RSpec/AnyInstanceOf
+
+      merge_request_metrics = table(:merge_request_metrics)
+
+      disable_migrations_output do
+        reversible_migration do |migration|
+          migration.before -> {
+            merge_request_metrics.reset_column_information
+
+            expect(merge_request_metrics.columns.find { |c| c.name == 'id' }.sql_type).to eq('integer')
+            expect(merge_request_metrics.columns.find do |c|
+                     c.name == 'id_convert_to_bigint'
+                   end.sql_type).to eq('bigint')
+          }
+
+          migration.after -> {
+            merge_request_metrics.reset_column_information
+
+            expect(merge_request_metrics.columns.find { |c| c.name == 'id' }.sql_type).to eq('integer')
+            expect(merge_request_metrics.columns.find do |c|
+                     c.name == 'id_convert_to_bigint'
+                   end.sql_type).to eq('bigint')
+          }
+        end
+      end
+    end
+  end
+end