diff options
| author | Yves Senn <yves.senn@gmail.com> | 2015-03-02 17:35:05 +0100 |
|---|---|---|
| committer | Yves Senn <yves.senn@gmail.com> | 2015-03-02 17:43:01 +0100 |
| commit | 3f4964299acd1351a519293991b97dd78b775073 (patch) | |
| tree | bc9312a08fae97e2b9d810c9d34653ffd0a62c15 /activerecord/test/cases/migrator_test.rb | |
| parent | e20dc1b313303540b44effd94fbc8cf39121ad28 (diff) | |
| download | rails-3f4964299acd1351a519293991b97dd78b775073.tar.gz rails-3f4964299acd1351a519293991b97dd78b775073.tar.bz2 rails-3f4964299acd1351a519293991b97dd78b775073.zip | |
tests, favor `drop_table` and `:if_exists` over raw SQL.
We've replaced most querues using DROP TABLE in our tests already.
This patch replaces the last couple calls.
Diffstat (limited to 'activerecord/test/cases/migrator_test.rb')
| -rw-r--r-- | activerecord/test/cases/migrator_test.rb | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/activerecord/test/cases/migrator_test.rb b/activerecord/test/cases/migrator_test.rb index c0daa83e9c..1760314099 100644 --- a/activerecord/test/cases/migrator_test.rb +++ b/activerecord/test/cases/migrator_test.rb @@ -312,7 +312,7 @@ class MigratorTest < ActiveRecord::TestCase def test_migrator_db_has_no_schema_migrations_table _, migrator = migrator_class(3) - ActiveRecord::Base.connection.execute("DROP TABLE schema_migrations") + ActiveRecord::Base.connection.drop_table "schema_migrations", if_exists: true assert_not ActiveRecord::Base.connection.table_exists?('schema_migrations') migrator.migrate("valid", 1) assert ActiveRecord::Base.connection.table_exists?('schema_migrations') |