Permalink
Browse files

Merge pull request #26271 from jcoleman/fix-unnecessary-query-cache-b…

…usting-test

Test that AR query cache isn't busted when types are not same object
  • Loading branch information...
guilleiguaran committed Aug 24, 2016
2 parents dd77e1b + 3df5173 commit ef8315ac784aa778b71c803a8aa018676880b396
Showing with 20 additions and 0 deletions.
  1. +20 −0 activerecord/test/cases/query_cache_test.rb
@@ -212,6 +212,26 @@ def test_query_cache_doesnt_leak_cached_results_of_rolled_back_queries
end
end
def test_query_cached_even_when_types_are_reset
Task.cache do
# Warm the cache
task = Task.find(1)
Task.connection.type_map.clear
# Preload the type cache again (so we don't have those queries issued during our assertions)
Task.connection.send(:initialize_type_map, Task.connection.type_map)
# Clear places where type information is cached
Task.reset_column_information
Task.initialize_find_by_cache
assert_queries(0) do
Task.find(1)
end
end
end
private
def middleware(&app)
executor = Class.new(ActiveSupport::Executor)

0 comments on commit ef8315a

Please sign in to comment.