From 1db7a5c285eeb61acc998c0c27788a61bd948d5c Mon Sep 17 00:00:00 2001 From: eileencodes Date: Mon, 7 Nov 2016 15:46:56 -0500 Subject: Don't load ActionSystemTest in production By moving to the TestUnit Railtie, and doing the file requirement inside the onload call we can avoid loading ActionSystemTest in production and load it in the test env. This is important for performance reasons - loading up unnecessary files and object is expensive, especially when they should never be used in production. --- actionsystemtest/lib/action_system_test.rb | 2 -- actionsystemtest/lib/action_system_test/railtie.rb | 17 ----------------- 2 files changed, 19 deletions(-) delete mode 100644 actionsystemtest/lib/action_system_test/railtie.rb (limited to 'actionsystemtest/lib') diff --git a/actionsystemtest/lib/action_system_test.rb b/actionsystemtest/lib/action_system_test.rb index 4a80e34695..18b992a3ca 100644 --- a/actionsystemtest/lib/action_system_test.rb +++ b/actionsystemtest/lib/action_system_test.rb @@ -50,8 +50,6 @@ require "action_system_test/driver_adapter" module ActionSystemTest include ActionSystemTest::TestHelper include ActionSystemTest::DriverAdapter - - ActiveSupport.run_load_hooks(:system_testing, self) end class ActionSystemTestCase < ActionDispatch::IntegrationTest diff --git a/actionsystemtest/lib/action_system_test/railtie.rb b/actionsystemtest/lib/action_system_test/railtie.rb deleted file mode 100644 index 6f4fe1b993..0000000000 --- a/actionsystemtest/lib/action_system_test/railtie.rb +++ /dev/null @@ -1,17 +0,0 @@ -require "action_system_test" - -module ActionSystemTest - # = System Testing Railtie - class Railtie < Rails::Railtie # :nodoc: - config.system_testing = ActiveSupport::OrderedOptions.new - - initializer "system_testing.set_configs" do |app| - options = app.config.system_testing - options.driver ||= ActionSystemTest.default_driver - - ActiveSupport.on_load(:system_testing) do - options.each { |k,v| send("#{k}=", v) } - end - end - end -end -- cgit v1.2.3