1 files changed, 31 insertions, 31 deletions
diff --git a/spec/lib/gitlab/database/migration_helpers/v2_spec.rb b/spec/lib/gitlab/database/migration_helpers/v2_spec.rb
index 854e97ef897..acf775b3538 100644
--- a/spec/lib/gitlab/database/migration_helpers/v2_spec.rb
+++ b/spec/lib/gitlab/database/migration_helpers/v2_spec.rb
@@ -20,7 +20,7 @@ RSpec.describe Gitlab::Database::MigrationHelpers::V2 do
     let(:model) { Class.new(ActiveRecord::Base) }
 
     before do
-      model.table_name = :test_table
+      model.table_name = :_test_table
     end
 
     context 'when called inside a transaction block' do
@@ -30,19 +30,19 @@ RSpec.describe Gitlab::Database::MigrationHelpers::V2 do
 
       it 'raises an error' do
         expect do
-          migration.public_send(operation, :test_table, :original, :renamed)
+          migration.public_send(operation, :_test_table, :original, :renamed)
         end.to raise_error("#{operation} can not be run inside a transaction")
       end
     end
 
     context 'when the existing column has a default value' do
       before do
-        migration.change_column_default :test_table, existing_column, 'default value'
+        migration.change_column_default :_test_table, existing_column, 'default value'
       end
 
       it 'raises an error' do
         expect do
-          migration.public_send(operation, :test_table, :original, :renamed)
+          migration.public_send(operation, :_test_table, :original, :renamed)
         end.to raise_error("#{operation} does not currently support columns with default values")
       end
     end
@@ -51,18 +51,18 @@ RSpec.describe Gitlab::Database::MigrationHelpers::V2 do
       context 'when the batch column does not exist' do
         it 'raises an error' do
           expect do
-            migration.public_send(operation, :test_table, :original, :renamed, batch_column_name: :missing)
-          end.to raise_error('Column missing does not exist on test_table')
+            migration.public_send(operation, :_test_table, :original, :renamed, batch_column_name: :missing)
+          end.to raise_error('Column missing does not exist on _test_table')
         end
       end
 
       context 'when the batch column does exist' do
         it 'passes it when creating the column' do
           expect(migration).to receive(:create_column_from)
-            .with(:test_table, existing_column, added_column, type: nil, batch_column_name: :status)
+            .with(:_test_table, existing_column, added_column, type: nil, batch_column_name: :status)
             .and_call_original
 
-          migration.public_send(operation, :test_table, :original, :renamed, batch_column_name: :status)
+          migration.public_send(operation, :_test_table, :original, :renamed, batch_column_name: :status)
         end
       end
     end
@@ -71,17 +71,17 @@ RSpec.describe Gitlab::Database::MigrationHelpers::V2 do
       existing_record_1 = model.create!(status: 0, existing_column => 'existing')
       existing_record_2 = model.create!(status: 0, existing_column => nil)
 
-      migration.send(operation, :test_table, :original, :renamed)
+      migration.send(operation, :_test_table, :original, :renamed)
       model.reset_column_information
 
-      expect(migration.column_exists?(:test_table, added_column)).to eq(true)
+      expect(migration.column_exists?(:_test_table, added_column)).to eq(true)
 
       expect(existing_record_1.reload).to have_attributes(status: 0, original: 'existing', renamed: 'existing')
       expect(existing_record_2.reload).to have_attributes(status: 0, original: nil, renamed: nil)
     end
 
     it 'installs triggers to sync new data' do
-      migration.public_send(operation, :test_table, :original, :renamed)
+      migration.public_send(operation, :_test_table, :original, :renamed)
       model.reset_column_information
 
       new_record_1 = model.create!(status: 1, original: 'first')
@@ -102,7 +102,7 @@ RSpec.describe Gitlab::Database::MigrationHelpers::V2 do
     before do
       allow(migration).to receive(:transaction_open?).and_return(false)
 
-      migration.create_table :test_table do |t|
+      migration.create_table :_test_table do |t|
         t.integer :status, null: false
         t.text :original
         t.text :other_column
@@ -118,8 +118,8 @@ RSpec.describe Gitlab::Database::MigrationHelpers::V2 do
     context 'when the column to rename does not exist' do
       it 'raises an error' do
         expect do
-          migration.rename_column_concurrently :test_table, :missing_column, :renamed
-        end.to raise_error('Column missing_column does not exist on test_table')
+          migration.rename_column_concurrently :_test_table, :missing_column, :renamed
+        end.to raise_error('Column missing_column does not exist on _test_table')
       end
     end
   end
@@ -128,7 +128,7 @@ RSpec.describe Gitlab::Database::MigrationHelpers::V2 do
     before do
       allow(migration).to receive(:transaction_open?).and_return(false)
 
-      migration.create_table :test_table do |t|
+      migration.create_table :_test_table do |t|
         t.integer :status, null: false
         t.text :other_column
         t.text :renamed
@@ -144,8 +144,8 @@ RSpec.describe Gitlab::Database::MigrationHelpers::V2 do
     context 'when the renamed column does not exist' do
       it 'raises an error' do
         expect do
-          migration.undo_cleanup_concurrent_column_rename :test_table, :original, :missing_column
-        end.to raise_error('Column missing_column does not exist on test_table')
+          migration.undo_cleanup_concurrent_column_rename :_test_table, :original, :missing_column
+        end.to raise_error('Column missing_column does not exist on _test_table')
       end
     end
   end
@@ -156,25 +156,25 @@ RSpec.describe Gitlab::Database::MigrationHelpers::V2 do
     before do
       allow(migration).to receive(:transaction_open?).and_return(false)
 
-      migration.create_table :test_table do |t|
+      migration.create_table :_test_table do |t|
         t.integer :status, null: false
         t.text :original
         t.text :other_column
       end
 
-      migration.rename_column_concurrently :test_table, :original, :renamed
+      migration.rename_column_concurrently :_test_table, :original, :renamed
     end
 
     context 'when the helper is called repeatedly' do
       before do
-        migration.public_send(operation, :test_table, :original, :renamed)
+        migration.public_send(operation, :_test_table, :original, :renamed)
       end
 
       it 'does not make repeated attempts to cleanup' do
         expect(migration).not_to receive(:remove_column)
 
         expect do
-          migration.public_send(operation, :test_table, :original, :renamed)
+          migration.public_send(operation, :_test_table, :original, :renamed)
         end.not_to raise_error
       end
     end
@@ -182,26 +182,26 @@ RSpec.describe Gitlab::Database::MigrationHelpers::V2 do
     context 'when the renamed column exists' do
       let(:triggers) do
         [
-          ['trigger_7cc71f92fd63', 'function_for_trigger_7cc71f92fd63', before: 'insert'],
-          ['trigger_f1a1f619636a', 'function_for_trigger_f1a1f619636a', before: 'update'],
-          ['trigger_769a49938884', 'function_for_trigger_769a49938884', before: 'update']
+          ['trigger_020dbcb8cdd0', 'function_for_trigger_020dbcb8cdd0', before: 'insert'],
+          ['trigger_6edaca641d03', 'function_for_trigger_6edaca641d03', before: 'update'],
+          ['trigger_a3fb9f3add34', 'function_for_trigger_a3fb9f3add34', before: 'update']
         ]
       end
 
       it 'removes the sync triggers and renamed columns' do
         triggers.each do |(trigger_name, function_name, event)|
           expect_function_to_exist(function_name)
-          expect_valid_function_trigger(:test_table, trigger_name, function_name, event)
+          expect_valid_function_trigger(:_test_table, trigger_name, function_name, event)
         end
 
-        expect(migration.column_exists?(:test_table, added_column)).to eq(true)
+        expect(migration.column_exists?(:_test_table, added_column)).to eq(true)
 
-        migration.public_send(operation, :test_table, :original, :renamed)
+        migration.public_send(operation, :_test_table, :original, :renamed)
 
-        expect(migration.column_exists?(:test_table, added_column)).to eq(false)
+        expect(migration.column_exists?(:_test_table, added_column)).to eq(false)
 
         triggers.each do |(trigger_name, function_name, _)|
-          expect_trigger_not_to_exist(:test_table, trigger_name)
+          expect_trigger_not_to_exist(:_test_table, trigger_name)
           expect_function_not_to_exist(function_name)
         end
       end
@@ -223,7 +223,7 @@ RSpec.describe Gitlab::Database::MigrationHelpers::V2 do
   end
 
   describe '#create_table' do
-    let(:table_name) { :test_table }
+    let(:table_name) { :_test_table }
     let(:column_attributes) do
       [
         { name: 'id',         sql_type: 'bigint',                   null: false, default: nil    },
@@ -245,7 +245,7 @@ RSpec.describe Gitlab::Database::MigrationHelpers::V2 do
         end
 
         expect_table_columns_to_match(column_attributes, table_name)
-        expect_check_constraint(table_name, 'check_cda6f69506', 'char_length(name) <= 100')
+        expect_check_constraint(table_name, 'check_e9982cf9da', 'char_length(name) <= 100')
       end
     end
   end