Add a test that exercice better the behavior we expect in the query cache

In production the query cache was already being loaded before the first
request even without #33856, so added a test to make sure of it.

This new test is passing even if #33856 is reverted.
上级 b79f3cc9
......@@ -371,7 +371,7 @@ def test_initialize_can_be_called_at_any_time
end
end
test "active record query cache hooks are installed before first request" do
test "active record query cache hooks are installed before first request in production" do
app_file "app/controllers/omg_controller.rb", <<-RUBY
begin
class OmgController < ActionController::Metal
......@@ -395,7 +395,40 @@ def show
end
RUBY
require "#{rails_root}/config/environment"
boot_app "production"
require "rack/test"
extend Rack::Test::Methods
get "/omg/show"
assert_equal "Query cache is enabled.", last_response.body
end
test "active record query cache hooks are installed before first request in development" do
app_file "app/controllers/omg_controller.rb", <<-RUBY
begin
class OmgController < ActionController::Metal
ActiveSupport.run_load_hooks(:action_controller, self)
def show
if ActiveRecord::Base.connection.query_cache_enabled
self.response_body = ["Query cache is enabled."]
else
self.response_body = ["Expected ActiveRecord::Base.connection.query_cache_enabled to be true"]
end
end
end
rescue => e
puts "Error loading metal: \#{e.class} \#{e.message}"
end
RUBY
app_file "config/routes.rb", <<-RUBY
Rails.application.routes.draw do
get "/:controller(/:action)"
end
RUBY
boot_app "development"
require "rack/test"
extend Rack::Test::Methods
......@@ -415,4 +448,12 @@ def setup_ar!
end
end
end
def boot_app(env = "development")
ENV["RAILS_ENV"] = env
require "#{app_path}/config/environment"
ensure
ENV.delete "RAILS_ENV"
end
end
Markdown is supported
0% .
You are about to add 0 people to the discussion. Proceed with caution.
先完成此消息的编辑!
想要评论请 注册