aboutsummaryrefslogtreecommitdiffstats
path: root/activerecord/test/cases
diff options
context:
space:
mode:
authorLachlan Sylvester <lachlan.sylvester@publicisfrontfoot.com.au>2016-01-05 15:40:34 +1100
committerLachlan Sylvester <lachlan.sylvester@hypothetical.com.au>2016-01-06 19:20:11 +1100
commit76dc41abdb33f69c078818dd8142bdf47834baa5 (patch)
treed5918f5db139421792cd1fdb51ba94b025dc2264 /activerecord/test/cases
parent0ff3e9466a4f476ee2e56d9e2b40acce01924683 (diff)
downloadrails-76dc41abdb33f69c078818dd8142bdf47834baa5.tar.gz
rails-76dc41abdb33f69c078818dd8142bdf47834baa5.tar.bz2
rails-76dc41abdb33f69c078818dd8142bdf47834baa5.zip
fix remove_index for postgresql when running legacy migrations
Diffstat (limited to 'activerecord/test/cases')
-rw-r--r--activerecord/test/cases/migration/compatibility_test.rb16
1 files changed, 16 insertions, 0 deletions
diff --git a/activerecord/test/cases/migration/compatibility_test.rb b/activerecord/test/cases/migration/compatibility_test.rb
index 267d2fcccc..b1e1d72944 100644
--- a/activerecord/test/cases/migration/compatibility_test.rb
+++ b/activerecord/test/cases/migration/compatibility_test.rb
@@ -21,6 +21,7 @@ module ActiveRecord
teardown do
connection.drop_table :testings rescue nil
ActiveRecord::Migration.verbose = @verbose_was
+ ActiveRecord::SchemaMigration.delete_all
end
def test_migration_doesnt_remove_named_index
@@ -37,6 +38,21 @@ module ActiveRecord
assert_raise(StandardError) { ActiveRecord::Migrator.new(:up, [migration]).migrate }
assert connection.index_exists?(:testings, :foo, name: "custom_index_name")
end
+
+ def test_migration_does_remove_unnamed_index
+ connection.add_index :testings, :bar
+
+ migration = Class.new(ActiveRecord::Migration[4.2]) {
+ def version; 101 end
+ def migrate(x)
+ remove_index :testings, :bar
+ end
+ }.new
+
+ assert connection.index_exists?(:testings, :bar)
+ ActiveRecord::Migrator.new(:up, [migration]).migrate
+ assert_not connection.index_exists?(:testings, :bar)
+ end
end
end
end