unify into ActionView::-scoped assertions

rails · Nov 18, 2023 · cda3a52 · cda3a52
1 parent 714c350
commit cda3a52
Show file tree

Hide file tree

Showing 8 changed files with 9 additions and 46 deletions.
diff --git a/actionpack/lib/action_controller/railtie.rb b/actionpack/lib/action_controller/railtie.rb
@@ -148,9 +148,9 @@ class Railtie < Rails::Railtie # :nodoc:
 
         case assertions
         when :capybara
-          include ActionDispatch::CapybaraAssertions
+          include ActionView::CapybaraAssertions
         when :rails_dom_testing
-          include ActionDispatch::RailsDomTestingAssertions
+          include ActionView::RailsDomTestingAssertions
         else
           raise ArgumentError.new("unrecognized value #{assertions.inspect} for config.action_controller.assertions")
         end

diff --git a/actionpack/lib/action_dispatch.rb b/actionpack/lib/action_dispatch.rb
@@ -128,8 +128,6 @@ def self.resolve_store(session_store) # :nodoc:
 
   autoload_under "testing" do
     autoload :Assertions
-    autoload :CapybaraAssertions
-    autoload :RailsDomTestingAssertions
     autoload :Integration
     autoload :IntegrationTest, "action_dispatch/testing/integration"
     autoload :TestProcess

diff --git a/actionpack/lib/action_dispatch/railtie.rb b/actionpack/lib/action_dispatch/railtie.rb
@@ -81,9 +81,9 @@ class Railtie < Rails::Railtie # :nodoc:
       ActiveSupport.on_load(:action_dispatch_integration_test) do
         case assertions
         when :capybara
-          include ActionDispatch::CapybaraAssertions
+          include ActionView::CapybaraAssertions
         when :rails_dom_testing
-          include ActionDispatch::RailsDomTestingAssertions
+          include ActionView::RailsDomTestingAssertions
         else
           raise ArgumentError.new("unrecognized value #{assertions.inspect} for config.action_dispatch.assertions")
         end

diff --git a/actionpack/lib/action_dispatch/testing/capybara_assertions.rb b/actionpack/lib/action_dispatch/testing/capybara_assertions.rb
diff --git a/actionpack/lib/action_dispatch/testing/rails_dom_testing_assertions.rb b/actionpack/lib/action_dispatch/testing/rails_dom_testing_assertions.rb
diff --git a/actionpack/test/abstract_unit.rb b/actionpack/test/abstract_unit.rb
@@ -104,11 +104,11 @@ def call(env)
 end
 
 ActiveSupport.on_load :action_controller_test_case do
-  include ActionDispatch::RailsDomTestingAssertions
+  include ActionView::RailsDomTestingAssertions
 end
 
 class ActionDispatch::IntegrationTest < ActiveSupport::TestCase
-  include ActionDispatch::RailsDomTestingAssertions
+  include ActionView::RailsDomTestingAssertions
 
   def self.build_app(routes = nil)
     RoutedRackApp.new(routes || ActionDispatch::Routing::RouteSet.new) do |middleware|

diff --git a/actionpack/test/dispatch/capybara_assertions_test.rb b/actionpack/test/dispatch/capybara_assertions_test.rb
@@ -9,7 +9,7 @@ def create
 end
 
 class ControllerCapybaraAssertionsTest < ActionController::TestCase
-  include ActionDispatch::CapybaraAssertions
+  include ActionView::CapybaraAssertions
 
   setup do
     @controller = RendersController.new
@@ -75,7 +75,7 @@ class ControllerCapybaraAssertionsTest < ActionController::TestCase
 end
 
 class IntegrationCapybaraAssertionsTest < ActionDispatch::IntegrationTest
-  include ActionDispatch::CapybaraAssertions
+  include ActionView::CapybaraAssertions
 
   APP = build_app(ActionDispatch::Routing::RouteSet.new.tap { |routes|
     routes.draw { post "/", to: "renders#create" }

diff --git a/railties/test/abstract_unit.rb b/railties/test/abstract_unit.rb
@@ -25,7 +25,7 @@ class ActiveSupport::TestCase
 end
 
 ActiveSupport.on_load(:action_controller_test_case) do
-  include ActionDispatch::RailsDomTestingAssertions
+  include ActionView::RailsDomTestingAssertions
 end
 
 require_relative "../../tools/test_common"