Restructure TemplateAssertions-related code to eliminate circular requires.

Also, no need to include dependencies in AS::Concerns inside included blocks.

2 files changed, 6 insertions, 14 deletions

diff --git a/actionpack/lib/action_dispatch/testing/assertions.rb b/actionpack/lib/action_dispatch/testing/assertions.rb
index 822150b768..226baf9ad0 100644
--- a/actionpack/lib/action_dispatch/testing/assertions.rb
+++ b/actionpack/lib/action_dispatch/testing/assertions.rb
@@ -8,12 +8,11 @@ module ActionDispatch
 
     extend ActiveSupport::Concern
 
-    included do
-      include DomAssertions
-      include ResponseAssertions
-      include RoutingAssertions
-      include SelectorAssertions
-      include TagAssertions
-    end
+    include DomAssertions
+    include ResponseAssertions
+    include RoutingAssertions
+    include SelectorAssertions
+    include TagAssertions
   end
 end
+
diff --git a/actionpack/lib/action_dispatch/testing/assertions/response.rb b/actionpack/lib/action_dispatch/testing/assertions/response.rb
index 3335742d47..606b01893e 100644
--- a/actionpack/lib/action_dispatch/testing/assertions/response.rb
+++ b/actionpack/lib/action_dispatch/testing/assertions/response.rb
@@ -6,13 +6,6 @@ module ActionDispatch
     module ResponseAssertions
       extend ActiveSupport::Concern
 
-      included do
-        # TODO: Need to pull in AV::Template monkey patches that track which
-        # templates are rendered. assert_template should probably be part
-        # of AV instead of AD.
-        require 'action_view/test_case'
-      end
-
       # Asserts that the response is one of the following types:
       #
       # * <tt>:success</tt>   - Status code was 200