mirror of
https://github.com/rails/rails.git
synced 2022-11-09 12:12:34 -05:00
Restructure TemplateAssertions-related code to eliminate circular requires.
Also, no need to include dependencies in AS::Concerns inside included blocks.
This commit is contained in:
parent
0afefa9504
commit
ddc584e89e
3 changed files with 13 additions and 20 deletions
|
@ -37,12 +37,13 @@ module ActionController
|
|||
autoload :UrlFor
|
||||
end
|
||||
|
||||
autoload :Integration, 'action_controller/deprecated/integration_test'
|
||||
autoload :IntegrationTest, 'action_controller/deprecated/integration_test'
|
||||
autoload :PerformanceTest, 'action_controller/deprecated/performance_test'
|
||||
autoload :UrlWriter, 'action_controller/deprecated'
|
||||
autoload :Routing, 'action_controller/deprecated'
|
||||
autoload :TestCase, 'action_controller/test_case'
|
||||
autoload :Integration, 'action_controller/deprecated/integration_test'
|
||||
autoload :IntegrationTest, 'action_controller/deprecated/integration_test'
|
||||
autoload :PerformanceTest, 'action_controller/deprecated/performance_test'
|
||||
autoload :UrlWriter, 'action_controller/deprecated'
|
||||
autoload :Routing, 'action_controller/deprecated'
|
||||
autoload :TestCase, 'action_controller/test_case'
|
||||
autoload :TemplateAssertions, 'action_controller/test_case'
|
||||
|
||||
eager_autoload do
|
||||
autoload :RecordIdentifier
|
||||
|
|
|
@ -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
|
||||
|
||||
|
|
|
@ -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
|
||||
|
|
Loading…
Reference in a new issue