diff options
| author | Jon Leighton <j@jonathanleighton.com> | 2011-12-13 23:43:42 +0000 |
|---|---|---|
| committer | Jon Leighton <j@jonathanleighton.com> | 2011-12-13 23:46:26 +0000 |
| commit | 38703ac8972c7e8f3f3f1ac95aa506cc4ae30ef0 (patch) | |
| tree | 5cc1bc40fa363bea7719e1e23ecdba8a8b3f945d /activerecord/test | |
| parent | c7552d988e5dec76d6c515a86f73701b647aa784 (diff) | |
| download | rails-38703ac8972c7e8f3f3f1ac95aa506cc4ae30ef0.tar.gz rails-38703ac8972c7e8f3f3f1ac95aa506cc4ae30ef0.tar.bz2 rails-38703ac8972c7e8f3f3f1ac95aa506cc4ae30ef0.zip | |
Revert naive O(1) table_exists? implementation.
It was a bad idea to rescue exceptions here. This can interfere with
transaction rollbacks which seems to be the cause of current CI
failure.
Instead, each adapter should implement its own DB-specific O(1)
implementation, and we fall back on the generic, slower, implementation
otherwise.
Diffstat (limited to 'activerecord/test')
| -rw-r--r-- | activerecord/test/cases/adapter_test.rb | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/activerecord/test/cases/adapter_test.rb b/activerecord/test/cases/adapter_test.rb index 5d5ff53004..56c359650e 100644 --- a/activerecord/test/cases/adapter_test.rb +++ b/activerecord/test/cases/adapter_test.rb @@ -17,6 +17,7 @@ module ActiveRecord def test_table_exists? assert @connection.table_exists?("accounts") assert !@connection.table_exists?("nonexistingtable") + assert !@connection.table_exists?(nil) end def test_indexes |