diff options
author | Sean Griffin <sean@seantheprogrammer.com> | 2016-06-29 09:46:45 -0400 |
---|---|---|
committer | Sean Griffin <sean@seantheprogrammer.com> | 2016-06-29 09:50:52 -0400 |
commit | 4565de0da14a4b3d75e8b1a652fcb6dcd947c451 (patch) | |
tree | 4d4d2bebc90c9fe78f5f3e834bc9072b982ec650 /activerecord/test/cases | |
parent | 10539ad7a998d305dd94d0ff3b39991ad3918c18 (diff) | |
download | rails-4565de0da14a4b3d75e8b1a652fcb6dcd947c451.tar.gz rails-4565de0da14a4b3d75e8b1a652fcb6dcd947c451.tar.bz2 rails-4565de0da14a4b3d75e8b1a652fcb6dcd947c451.zip |
Remove unused `ActiveRecord::Base.connection_id`
This method appears to have been partially used in connection pool
caching, but it was introduced without much reasoning or any tests. One
edge case test was added later on, but it was focused on implementation
details. This method is no longer used outside of tests, and as such is
removed.
Diffstat (limited to 'activerecord/test/cases')
-rw-r--r-- | activerecord/test/cases/query_cache_test.rb | 12 |
1 files changed, 0 insertions, 12 deletions
diff --git a/activerecord/test/cases/query_cache_test.rb b/activerecord/test/cases/query_cache_test.rb index d5c01315c1..03ec063671 100644 --- a/activerecord/test/cases/query_cache_test.rb +++ b/activerecord/test/cases/query_cache_test.rb @@ -39,18 +39,6 @@ class QueryCacheTest < ActiveRecord::TestCase assert ActiveRecord::Base.connection.query_cache_enabled, 'cache on' end - def test_exceptional_middleware_assigns_original_connection_id_on_error - connection_id = ActiveRecord::Base.connection_id - - mw = middleware { |env| - ActiveRecord::Base.connection_id = self.object_id - raise "lol borked" - } - assert_raises(RuntimeError) { mw.call({}) } - - assert_equal connection_id, ActiveRecord::Base.connection_id - end - def test_middleware_delegates called = false mw = middleware { |env| |