diff options
author | Seva Orlov <vseorlov@gmail.com> | 2016-04-11 20:57:19 +0300 |
---|---|---|
committer | Jeremy Daer <jeremydaer@gmail.com> | 2016-04-24 14:32:55 -0700 |
commit | c41ef01aec40bfaa9af707551d1e8a1f9f7380d1 (patch) | |
tree | 1bb0e0172a8da1110351499cbac01db4f8738973 /activerecord/test/cases/adapters/postgresql | |
parent | 6c6eeda54d0f2c8e689772b50cc5c4419fa82749 (diff) | |
download | rails-c41ef01aec40bfaa9af707551d1e8a1f9f7380d1.tar.gz rails-c41ef01aec40bfaa9af707551d1e8a1f9f7380d1.tar.bz2 rails-c41ef01aec40bfaa9af707551d1e8a1f9f7380d1.zip |
remove_index do not fetch indexes if name is specified
There is no need to fetch all table indexes in remove_index if name is specified. If name is wrong, then StatementInvalid will be raised.
Signed-off-by: Jeremy Daer <jeremydaer@gmail.com>
Diffstat (limited to 'activerecord/test/cases/adapters/postgresql')
-rw-r--r-- | activerecord/test/cases/adapters/postgresql/active_schema_test.rb | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/activerecord/test/cases/adapters/postgresql/active_schema_test.rb b/activerecord/test/cases/adapters/postgresql/active_schema_test.rb index ed44bf7362..edb391b525 100644 --- a/activerecord/test/cases/adapters/postgresql/active_schema_test.rb +++ b/activerecord/test/cases/adapters/postgresql/active_schema_test.rb @@ -69,6 +69,11 @@ class PostgresqlActiveSchemaTest < ActiveRecord::PostgreSQLTestCase ActiveRecord::ConnectionAdapters::PostgreSQLAdapter.send :remove_method, :index_name_for_remove end + def test_remove_index_when_name_is_specified + expected = %(DROP INDEX CONCURRENTLY "index_people_on_last_name") + assert_equal expected, remove_index(:people, name: "index_people_on_last_name", algorithm: :concurrently) + end + private def method_missing(method_symbol, *arguments) ActiveRecord::Base.connection.send(method_symbol, *arguments) |