Permalink
Browse files

Remove unnecessary system test code

It turns out that we don't need to require system tests in the railties
test helper so we can remove it. If you're using system tests they will
be loaded by inheriting from ActionDispatch::SystemTestCase and the
routes will be loaded by ActionDispatch::IntegrationTest.
  • Loading branch information...
eileencodes committed Mar 3, 2017
1 parent 76be78c commit d3e7d91050da858bd5ec6a061b96466bd9ac85ef
Showing with 0 additions and 13 deletions.
  1. +0 −13 railties/lib/rails/test_help.rb
@@ -11,10 +11,6 @@
require "active_support/testing/autorun"
-if defined?(Capybara) && defined?(Puma)
- require "action_dispatch/system_test_case"
-end
-
if defined?(ActiveRecord::Base)
ActiveRecord::Migration.maintain_test_schema!
@@ -48,12 +44,3 @@ def before_setup # :nodoc:
super
end
end
-
-if defined?(Capybara) && defined?(Puma)
- class ActionDispatch::SystemTestCase
- def before_setup # :nodoc:
- @routes = Rails.application.routes
- super
- end
- end
-end

0 comments on commit d3e7d91

Please sign in to comment.