Removed deprecated RouteSet API, still many tests fail
This commit is contained in:
parent
91fec0d24d
commit
b3eb26a161
@ -264,7 +264,6 @@ module ActionDispatch
|
|||||||
# Target specific controllers by prefixing the command with <tt>CONTROLLER=x</tt>.
|
# Target specific controllers by prefixing the command with <tt>CONTROLLER=x</tt>.
|
||||||
#
|
#
|
||||||
module Routing
|
module Routing
|
||||||
autoload :DeprecatedMapper, 'action_dispatch/routing/deprecated_mapper'
|
|
||||||
autoload :Mapper, 'action_dispatch/routing/mapper'
|
autoload :Mapper, 'action_dispatch/routing/mapper'
|
||||||
autoload :Route, 'action_dispatch/routing/route'
|
autoload :Route, 'action_dispatch/routing/route'
|
||||||
autoload :RouteSet, 'action_dispatch/routing/route_set'
|
autoload :RouteSet, 'action_dispatch/routing/route_set'
|
||||||
|
@ -443,11 +443,6 @@ def scope(*args)
|
|||||||
options = args.extract_options!
|
options = args.extract_options!
|
||||||
options = options.dup
|
options = options.dup
|
||||||
|
|
||||||
if name_prefix = options.delete(:name_prefix)
|
|
||||||
options[:as] ||= name_prefix
|
|
||||||
ActiveSupport::Deprecation.warn ":name_prefix was deprecated in the new router syntax. Use :as instead.", caller
|
|
||||||
end
|
|
||||||
|
|
||||||
options[:path] = args.first if args.first.is_a?(String)
|
options[:path] = args.first if args.first.is_a?(String)
|
||||||
recover = {}
|
recover = {}
|
||||||
|
|
||||||
@ -770,7 +765,7 @@ def resources(*resources, &block)
|
|||||||
end
|
end
|
||||||
|
|
||||||
resource_scope(Resource.new(resources.pop, options)) do
|
resource_scope(Resource.new(resources.pop, options)) do
|
||||||
yield if block_given?
|
instance_eval(&block) if block_given?
|
||||||
|
|
||||||
collection_scope do
|
collection_scope do
|
||||||
get :index if parent_resource.actions.include?(:index)
|
get :index if parent_resource.actions.include?(:index)
|
||||||
|
@ -1,7 +1,6 @@
|
|||||||
require 'rack/mount'
|
require 'rack/mount'
|
||||||
require 'forwardable'
|
require 'forwardable'
|
||||||
require 'active_support/core_ext/object/to_query'
|
require 'active_support/core_ext/object/to_query'
|
||||||
require 'action_dispatch/routing/deprecated_mapper'
|
|
||||||
|
|
||||||
module ActionDispatch
|
module ActionDispatch
|
||||||
module Routing
|
module Routing
|
||||||
@ -211,7 +210,6 @@ def initialize(request_class = ActionDispatch::Request)
|
|||||||
self.routes = []
|
self.routes = []
|
||||||
self.named_routes = NamedRouteCollection.new
|
self.named_routes = NamedRouteCollection.new
|
||||||
self.resources_path_names = self.class.default_resources_path_names.dup
|
self.resources_path_names = self.class.default_resources_path_names.dup
|
||||||
self.controller_namespaces = Set.new
|
|
||||||
self.default_url_options = {}
|
self.default_url_options = {}
|
||||||
|
|
||||||
self.request_class = request_class
|
self.request_class = request_class
|
||||||
@ -227,14 +225,10 @@ def draw(&block)
|
|||||||
clear! unless @disable_clear_and_finalize
|
clear! unless @disable_clear_and_finalize
|
||||||
|
|
||||||
mapper = Mapper.new(self)
|
mapper = Mapper.new(self)
|
||||||
if block.arity == 1
|
if default_scope
|
||||||
mapper.instance_exec(DeprecatedMapper.new(self), &block)
|
mapper.with_default_scope(default_scope, &block)
|
||||||
else
|
else
|
||||||
if default_scope
|
mapper.instance_exec(&block)
|
||||||
mapper.with_default_scope(default_scope, &block)
|
|
||||||
else
|
|
||||||
mapper.instance_exec(&block)
|
|
||||||
end
|
|
||||||
end
|
end
|
||||||
|
|
||||||
finalize! unless @disable_clear_and_finalize
|
finalize! unless @disable_clear_and_finalize
|
||||||
|
@ -49,14 +49,6 @@
|
|||||||
module Rails
|
module Rails
|
||||||
end
|
end
|
||||||
|
|
||||||
# Monkey patch the old routes initialization to be silenced.
|
|
||||||
class ActionDispatch::Routing::DeprecatedMapper
|
|
||||||
def initialize_with_silencer(*args)
|
|
||||||
ActiveSupport::Deprecation.silence { initialize_without_silencer(*args) }
|
|
||||||
end
|
|
||||||
alias_method_chain :initialize, :silencer
|
|
||||||
end
|
|
||||||
|
|
||||||
ActiveSupport::Dependencies.hook!
|
ActiveSupport::Dependencies.hook!
|
||||||
|
|
||||||
# Show backtraces for deprecated behavior for quicker cleanup.
|
# Show backtraces for deprecated behavior for quicker cleanup.
|
||||||
@ -128,14 +120,12 @@ class TestCase
|
|||||||
# Hold off drawing routes until all the possible controller classes
|
# Hold off drawing routes until all the possible controller classes
|
||||||
# have been loaded.
|
# have been loaded.
|
||||||
setup_once do
|
setup_once do
|
||||||
SharedTestRoutes.draw do |map|
|
SharedTestRoutes.draw do
|
||||||
# FIXME: match ':controller(/:action(/:id))'
|
match ':controller(/:action)'
|
||||||
map.connect ':controller/:action/:id'
|
|
||||||
end
|
end
|
||||||
|
|
||||||
ActionController::IntegrationTest.app.routes.draw do |map|
|
ActionController::IntegrationTest.app.routes.draw do
|
||||||
# FIXME: match ':controller(/:action(/:id))'
|
match ':controller(/:action)'
|
||||||
map.connect ':controller/:action/:id'
|
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
|
@ -198,7 +198,7 @@ def test_allows_session_fixation
|
|||||||
|
|
||||||
def with_test_route_set(options = {})
|
def with_test_route_set(options = {})
|
||||||
with_routing do |set|
|
with_routing do |set|
|
||||||
set.draw do |map|
|
set.draw do
|
||||||
match ':action', :to => 'active_record_store_test/test'
|
match ':action', :to => 'active_record_store_test/test'
|
||||||
end
|
end
|
||||||
|
|
||||||
|
@ -451,18 +451,18 @@ def with_namespaced_routes(name)
|
|||||||
|
|
||||||
def with_test_routes(options = {})
|
def with_test_routes(options = {})
|
||||||
with_routing do |set|
|
with_routing do |set|
|
||||||
set.draw do |map|
|
set.draw do
|
||||||
map.resources :projects do |projects|
|
resources :projects do
|
||||||
projects.resources :tasks
|
resources :tasks
|
||||||
projects.resource :bid do |bid|
|
resource :bid do
|
||||||
bid.resources :tasks
|
resources :tasks
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
map.resources :taxes do |taxes|
|
resources :taxes do
|
||||||
taxes.resources :faxes
|
resources :faxes
|
||||||
taxes.resource :bid
|
resource :bid
|
||||||
end
|
end
|
||||||
map.resources :series
|
resources :series
|
||||||
end
|
end
|
||||||
|
|
||||||
self.class.send(:include, @routes.url_helpers)
|
self.class.send(:include, @routes.url_helpers)
|
||||||
|
@ -222,7 +222,7 @@ def test_post
|
|||||||
# test the redirection to a named route
|
# test the redirection to a named route
|
||||||
def test_assert_redirect_to_named_route
|
def test_assert_redirect_to_named_route
|
||||||
with_routing do |set|
|
with_routing do |set|
|
||||||
set.draw do |map|
|
set.draw do
|
||||||
match 'route_one', :to => 'action_pack_assertions#nothing', :as => :route_one
|
match 'route_one', :to => 'action_pack_assertions#nothing', :as => :route_one
|
||||||
match ':controller/:action'
|
match ':controller/:action'
|
||||||
end
|
end
|
||||||
@ -236,7 +236,7 @@ def test_assert_redirect_to_named_route
|
|||||||
|
|
||||||
def test_assert_redirect_to_named_route_failure
|
def test_assert_redirect_to_named_route_failure
|
||||||
with_routing do |set|
|
with_routing do |set|
|
||||||
set.draw do |map|
|
set.draw do
|
||||||
match 'route_one', :to => 'action_pack_assertions#nothing', :as => :route_one
|
match 'route_one', :to => 'action_pack_assertions#nothing', :as => :route_one
|
||||||
match 'route_two', :to => 'action_pack_assertions#nothing', :id => 'two', :as => :route_two
|
match 'route_two', :to => 'action_pack_assertions#nothing', :id => 'two', :as => :route_two
|
||||||
match ':controller/:action'
|
match ':controller/:action'
|
||||||
@ -258,10 +258,9 @@ def test_assert_redirect_to_nested_named_route
|
|||||||
@controller = Admin::InnerModuleController.new
|
@controller = Admin::InnerModuleController.new
|
||||||
|
|
||||||
with_routing do |set|
|
with_routing do |set|
|
||||||
set.draw do |map|
|
set.draw do
|
||||||
match 'admin/inner_module', :to => 'admin/inner_module#index', :as => :admin_inner_module
|
match 'admin/inner_module', :to => 'admin/inner_module#index', :as => :admin_inner_module
|
||||||
# match ':controller/:action'
|
match ':controller/:action'
|
||||||
map.connect ':controller/:action/:id'
|
|
||||||
end
|
end
|
||||||
process :redirect_to_index
|
process :redirect_to_index
|
||||||
# redirection is <{"action"=>"index", "controller"=>"admin/admin/inner_module"}>
|
# redirection is <{"action"=>"index", "controller"=>"admin/admin/inner_module"}>
|
||||||
@ -273,10 +272,9 @@ def test_assert_redirected_to_top_level_named_route_from_nested_controller
|
|||||||
@controller = Admin::InnerModuleController.new
|
@controller = Admin::InnerModuleController.new
|
||||||
|
|
||||||
with_routing do |set|
|
with_routing do |set|
|
||||||
set.draw do |map|
|
set.draw do
|
||||||
match '/action_pack_assertions/:id', :to => 'action_pack_assertions#index', :as => :top_level
|
match '/action_pack_assertions/:id', :to => 'action_pack_assertions#index', :as => :top_level
|
||||||
# match ':controller/:action'
|
match ':controller/:action'
|
||||||
map.connect ':controller/:action/:id'
|
|
||||||
end
|
end
|
||||||
process :redirect_to_top_level_named_route
|
process :redirect_to_top_level_named_route
|
||||||
# assert_redirected_to "http://test.host/action_pack_assertions/foo" would pass because of exact match early return
|
# assert_redirected_to "http://test.host/action_pack_assertions/foo" would pass because of exact match early return
|
||||||
@ -288,11 +286,10 @@ def test_assert_redirected_to_top_level_named_route_with_same_controller_name_in
|
|||||||
@controller = Admin::InnerModuleController.new
|
@controller = Admin::InnerModuleController.new
|
||||||
|
|
||||||
with_routing do |set|
|
with_routing do |set|
|
||||||
set.draw do |map|
|
set.draw do
|
||||||
# this controller exists in the admin namespace as well which is the only difference from previous test
|
# this controller exists in the admin namespace as well which is the only difference from previous test
|
||||||
match '/user/:id', :to => 'user#index', :as => :top_level
|
match '/user/:id', :to => 'user#index', :as => :top_level
|
||||||
# match ':controller/:action'
|
match ':controller/:action'
|
||||||
map.connect ':controller/:action/:id'
|
|
||||||
end
|
end
|
||||||
process :redirect_to_top_level_named_route
|
process :redirect_to_top_level_named_route
|
||||||
# assert_redirected_to top_level_url('foo') would pass because of exact match early return
|
# assert_redirected_to top_level_url('foo') would pass because of exact match early return
|
||||||
|
@ -294,7 +294,7 @@ def test_named_routes_with_path_without_doing_a_request_first
|
|||||||
@controller.request = @request
|
@controller.request = @request
|
||||||
|
|
||||||
with_routing do |set|
|
with_routing do |set|
|
||||||
set.draw do |map|
|
set.draw do
|
||||||
resources :things
|
resources :things
|
||||||
end
|
end
|
||||||
|
|
||||||
|
@ -76,7 +76,7 @@ def teardown
|
|||||||
|
|
||||||
def test_page_caching_resources_saves_to_correct_path_with_extension_even_if_default_route
|
def test_page_caching_resources_saves_to_correct_path_with_extension_even_if_default_route
|
||||||
with_routing do |set|
|
with_routing do |set|
|
||||||
set.draw do |map|
|
set.draw do
|
||||||
match 'posts.:format', :to => 'posts#index', :as => :formatted_posts
|
match 'posts.:format', :to => 'posts#index', :as => :formatted_posts
|
||||||
match '/', :to => 'posts#index', :as => :main
|
match '/', :to => 'posts#index', :as => :main
|
||||||
end
|
end
|
||||||
@ -452,7 +452,7 @@ def test_forbidden_is_not_cached
|
|||||||
|
|
||||||
def test_xml_version_of_resource_is_treated_as_different_cache
|
def test_xml_version_of_resource_is_treated_as_different_cache
|
||||||
with_routing do |set|
|
with_routing do |set|
|
||||||
set.draw do |map|
|
set.draw do
|
||||||
match ':controller(/:action(.:format))'
|
match ':controller(/:action(.:format))'
|
||||||
end
|
end
|
||||||
|
|
||||||
|
@ -255,7 +255,7 @@ def get(path, parameters = nil, env = {})
|
|||||||
|
|
||||||
def with_test_route_set
|
def with_test_route_set
|
||||||
with_routing do |set|
|
with_routing do |set|
|
||||||
set.draw do |map|
|
set.draw do
|
||||||
match ':action', :to => FlashIntegrationTest::TestController
|
match ':action', :to => FlashIntegrationTest::TestController
|
||||||
end
|
end
|
||||||
|
|
||||||
|
@ -427,7 +427,7 @@ def with_test_route_set
|
|||||||
include set.url_helpers
|
include set.url_helpers
|
||||||
end
|
end
|
||||||
|
|
||||||
set.draw do |map|
|
set.draw do
|
||||||
match ':action', :to => controller
|
match ':action', :to => controller
|
||||||
get 'get/:action', :to => controller
|
get 'get/:action', :to => controller
|
||||||
end
|
end
|
||||||
|
@ -876,7 +876,7 @@ def test_error_is_raised_if_no_respond_to_is_declared_and_respond_with_is_called
|
|||||||
private
|
private
|
||||||
def with_test_route_set
|
def with_test_route_set
|
||||||
with_routing do |set|
|
with_routing do |set|
|
||||||
set.draw do |map|
|
set.draw do
|
||||||
resources :customers
|
resources :customers
|
||||||
resources :quiz_stores do
|
resources :quiz_stores do
|
||||||
resources :customers
|
resources :customers
|
||||||
|
@ -83,6 +83,9 @@ class RenderActionTest < Rack::TestCase
|
|||||||
end
|
end
|
||||||
|
|
||||||
class RenderLayoutTest < Rack::TestCase
|
class RenderLayoutTest < Rack::TestCase
|
||||||
|
def setup
|
||||||
|
end
|
||||||
|
|
||||||
describe "Both <controller_path>.html.erb and application.html.erb are missing"
|
describe "Both <controller_path>.html.erb and application.html.erb are missing"
|
||||||
|
|
||||||
test "rendering with layout => true" do
|
test "rendering with layout => true" do
|
||||||
|
@ -232,7 +232,7 @@ def test_redirect_to_back_with_no_referer
|
|||||||
|
|
||||||
def test_redirect_to_record
|
def test_redirect_to_record
|
||||||
with_routing do |set|
|
with_routing do |set|
|
||||||
set.draw do |map|
|
set.draw do
|
||||||
resources :workshops
|
resources :workshops
|
||||||
match ':controller/:action'
|
match ':controller/:action'
|
||||||
end
|
end
|
||||||
|
@ -1120,7 +1120,7 @@ def test_head_with_location_header
|
|||||||
|
|
||||||
def test_head_with_location_object
|
def test_head_with_location_object
|
||||||
with_routing do |set|
|
with_routing do |set|
|
||||||
set.draw do |map|
|
set.draw do
|
||||||
resources :customers
|
resources :customers
|
||||||
match ':controller/:action'
|
match ':controller/:action'
|
||||||
end
|
end
|
||||||
|
@ -70,10 +70,9 @@ def test_rendering_xml_should_call_to_xml_with_extra_options
|
|||||||
|
|
||||||
def test_rendering_with_object_location_should_set_header_with_url_for
|
def test_rendering_with_object_location_should_set_header_with_url_for
|
||||||
with_routing do |set|
|
with_routing do |set|
|
||||||
set.draw do |map|
|
set.draw do
|
||||||
resources :customers
|
resources :customers
|
||||||
# match ':controller/:action'
|
match ':controller/:action'
|
||||||
map.connect ':controller/:action/:id'
|
|
||||||
end
|
end
|
||||||
|
|
||||||
get :render_with_object_location
|
get :render_with_object_location
|
||||||
|
@ -371,7 +371,7 @@ def show_errors(exception)
|
|||||||
private
|
private
|
||||||
def with_test_routing
|
def with_test_routing
|
||||||
with_routing do |set|
|
with_routing do |set|
|
||||||
set.draw do |map|
|
set.draw do
|
||||||
match 'foo', :to => ::RescueTest::TestController.action(:foo)
|
match 'foo', :to => ::RescueTest::TestController.action(:foo)
|
||||||
match 'invalid', :to => ::RescueTest::TestController.action(:invalid)
|
match 'invalid', :to => ::RescueTest::TestController.action(:invalid)
|
||||||
match 'b00m', :to => ::RescueTest::TestController.action(:b00m)
|
match 'b00m', :to => ::RescueTest::TestController.action(:b00m)
|
||||||
|
@ -1,5 +1,6 @@
|
|||||||
require 'abstract_unit'
|
require 'abstract_unit'
|
||||||
require 'active_support/core_ext/object/try'
|
require 'active_support/core_ext/object/try'
|
||||||
|
require 'active_support/core_ext/object/with_options'
|
||||||
|
|
||||||
class ResourcesController < ActionController::Base
|
class ResourcesController < ActionController::Base
|
||||||
def index() render :nothing => true end
|
def index() render :nothing => true end
|
||||||
@ -32,36 +33,6 @@ class ImagesController < ResourcesController; end
|
|||||||
end
|
end
|
||||||
|
|
||||||
class ResourcesTest < ActionController::TestCase
|
class ResourcesTest < ActionController::TestCase
|
||||||
def test_should_arrange_actions
|
|
||||||
resource = ActionDispatch::Routing::DeprecatedMapper::Resource.new(:messages, {
|
|
||||||
:collection => { :rss => :get, :reorder => :post, :csv => :post },
|
|
||||||
:member => { :rss => :get, :atom => :get, :upload => :post, :fix => :post },
|
|
||||||
:new => { :preview => :get, :draft => :get }}, {})
|
|
||||||
|
|
||||||
assert_resource_methods [:rss], resource, :collection, :get
|
|
||||||
assert_resource_methods [:csv, :reorder], resource, :collection, :post
|
|
||||||
assert_resource_methods [:edit, :rss, :atom], resource, :member, :get
|
|
||||||
assert_resource_methods [:upload, :fix], resource, :member, :post
|
|
||||||
assert_resource_methods [:new, :preview, :draft], resource, :new, :get
|
|
||||||
end
|
|
||||||
|
|
||||||
def test_should_resource_controller_name_equal_resource_name_by_default
|
|
||||||
resource = ActionDispatch::Routing::DeprecatedMapper::Resource.new(:messages, {}, {})
|
|
||||||
assert_equal 'messages', resource.controller
|
|
||||||
end
|
|
||||||
|
|
||||||
def test_should_resource_controller_name_equal_controller_option
|
|
||||||
resource = ActionDispatch::Routing::DeprecatedMapper::Resource.new(:messages, {:controller => 'posts'}, {})
|
|
||||||
assert_equal 'posts', resource.controller
|
|
||||||
end
|
|
||||||
|
|
||||||
def test_should_all_singleton_paths_be_the_same
|
|
||||||
[ :path, :nesting_path_prefix, :member_path ].each do |method|
|
|
||||||
resource = ActionDispatch::Routing::DeprecatedMapper::SingletonResource.new(:messages, {:path_prefix => 'admin'}, {})
|
|
||||||
assert_equal 'admin/messages', resource.send(method)
|
|
||||||
end
|
|
||||||
end
|
|
||||||
|
|
||||||
def test_default_restful_routes
|
def test_default_restful_routes
|
||||||
with_restful_routing :messages do
|
with_restful_routing :messages do
|
||||||
assert_simply_restful_for :messages
|
assert_simply_restful_for :messages
|
||||||
@ -69,9 +40,9 @@ def test_default_restful_routes
|
|||||||
end
|
end
|
||||||
|
|
||||||
def test_override_paths_for_member_and_collection_methods
|
def test_override_paths_for_member_and_collection_methods
|
||||||
collection_methods = { 'rss' => :get, 'reorder' => :post, 'csv' => :post }
|
collection_methods = { :rss => :get, :reorder => :post, :csv => :post }
|
||||||
member_methods = { 'rss' => :get, :atom => :get, :upload => :post, :fix => :post }
|
member_methods = { :rss => :get, :atom => :get, :upload => :post, :fix => :post }
|
||||||
path_names = {:new => 'nuevo', 'rss' => 'canal', :fix => 'corrigir' }
|
path_names = {:new => 'nuevo', :rss => 'canal', :fix => 'corrigir' }
|
||||||
|
|
||||||
with_restful_routing :messages,
|
with_restful_routing :messages,
|
||||||
:collection => collection_methods,
|
:collection => collection_methods,
|
||||||
@ -89,7 +60,7 @@ def test_override_paths_for_member_and_collection_methods
|
|||||||
end
|
end
|
||||||
|
|
||||||
collection_methods.each do |action, method|
|
collection_methods.each do |action, method|
|
||||||
assert_recognizes(options.merge(:action => action),
|
assert_recognizes(options.merge(:action => action.to_s),
|
||||||
:path => "/messages/#{path_names[action] || action}",
|
:path => "/messages/#{path_names[action] || action}",
|
||||||
:method => method)
|
:method => method)
|
||||||
end
|
end
|
||||||
@ -112,12 +83,6 @@ def test_override_paths_for_member_and_collection_methods
|
|||||||
end
|
end
|
||||||
end
|
end
|
||||||
|
|
||||||
def test_override_paths_for_default_restful_actions
|
|
||||||
resource = ActionDispatch::Routing::DeprecatedMapper::Resource.new(:messages, {
|
|
||||||
:path_names => {:new => 'nuevo', :edit => 'editar'}}, {})
|
|
||||||
assert_equal resource.new_path, "#{resource.path}/nuevo"
|
|
||||||
end
|
|
||||||
|
|
||||||
def test_multiple_default_restful_routes
|
def test_multiple_default_restful_routes
|
||||||
with_restful_routing :messages, :comments do
|
with_restful_routing :messages, :comments do
|
||||||
assert_simply_restful_for :messages
|
assert_simply_restful_for :messages
|
||||||
@ -131,7 +96,7 @@ def test_with_custom_conditions
|
|||||||
end
|
end
|
||||||
end
|
end
|
||||||
|
|
||||||
def test_irregular_id_with_no_requirements_should_raise_error
|
def test_irregular_id_with_no_constraints_should_raise_error
|
||||||
expected_options = {:controller => 'messages', :action => 'show', :id => '1.1.1'}
|
expected_options = {:controller => 'messages', :action => 'show', :id => '1.1.1'}
|
||||||
|
|
||||||
with_restful_routing :messages do
|
with_restful_routing :messages do
|
||||||
@ -141,25 +106,25 @@ def test_irregular_id_with_no_requirements_should_raise_error
|
|||||||
end
|
end
|
||||||
end
|
end
|
||||||
|
|
||||||
def test_irregular_id_with_requirements_should_pass
|
def test_irregular_id_with_constraints_should_pass
|
||||||
expected_options = {:controller => 'messages', :action => 'show', :id => '1.1.1'}
|
expected_options = {:controller => 'messages', :action => 'show', :id => '1.1.1'}
|
||||||
|
|
||||||
with_restful_routing(:messages, :requirements => {:id => /[0-9]\.[0-9]\.[0-9]/}) do
|
with_restful_routing(:messages, :constraints => {:id => /[0-9]\.[0-9]\.[0-9]/}) do
|
||||||
assert_recognizes(expected_options, :path => 'messages/1.1.1', :method => :get)
|
assert_recognizes(expected_options, :path => 'messages/1.1.1', :method => :get)
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
|
|
||||||
def test_with_path_prefix_requirements
|
def test_with_path_prefix_constraints
|
||||||
expected_options = {:controller => 'messages', :action => 'show', :thread_id => '1.1.1', :id => '1'}
|
expected_options = {:controller => 'messages', :action => 'show', :thread_id => '1.1.1', :id => '1'}
|
||||||
with_restful_routing :messages, :path_prefix => '/thread/:thread_id', :requirements => {:thread_id => /[0-9]\.[0-9]\.[0-9]/} do
|
with_restful_routing :messages, :path_prefix => '/thread/:thread_id', :constraints => {:thread_id => /[0-9]\.[0-9]\.[0-9]/} do
|
||||||
assert_recognizes(expected_options, :path => 'thread/1.1.1/messages/1', :method => :get)
|
assert_recognizes(expected_options, :path => 'thread/1.1.1/messages/1', :method => :get)
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
|
|
||||||
def test_irregular_id_requirements_should_get_passed_to_member_actions
|
def test_irregular_id_constraints_should_get_passed_to_member_actions
|
||||||
expected_options = {:controller => 'messages', :action => 'custom', :id => '1.1.1'}
|
expected_options = {:controller => 'messages', :action => 'custom', :id => '1.1.1'}
|
||||||
|
|
||||||
with_restful_routing(:messages, :member => {:custom => :get}, :requirements => {:id => /[0-9]\.[0-9]\.[0-9]/}) do
|
with_restful_routing(:messages, :member => {:custom => :get}, :constraints => {:id => /[0-9]\.[0-9]\.[0-9]/}) do
|
||||||
assert_recognizes(expected_options, :path => 'messages/1.1.1/custom', :method => :get)
|
assert_recognizes(expected_options, :path => 'messages/1.1.1/custom', :method => :get)
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
@ -274,7 +239,7 @@ def test_with_member_action
|
|||||||
def test_with_member_action_and_requirement
|
def test_with_member_action_and_requirement
|
||||||
expected_options = {:controller => 'messages', :action => 'mark', :id => '1.1.1'}
|
expected_options = {:controller => 'messages', :action => 'mark', :id => '1.1.1'}
|
||||||
|
|
||||||
with_restful_routing(:messages, :requirements => {:id => /[0-9]\.[0-9]\.[0-9]/}, :member => { :mark => :get }) do
|
with_restful_routing(:messages, :constraints => {:id => /[0-9]\.[0-9]\.[0-9]/}, :member => { :mark => :get }) do
|
||||||
assert_recognizes(expected_options, :path => 'messages/1.1.1/mark', :method => :get)
|
assert_recognizes(expected_options, :path => 'messages/1.1.1/mark', :method => :get)
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
@ -411,10 +376,10 @@ def test_override_new_method
|
|||||||
|
|
||||||
def test_nested_restful_routes
|
def test_nested_restful_routes
|
||||||
with_routing do |set|
|
with_routing do |set|
|
||||||
set.draw do |map|
|
set.draw do
|
||||||
map.resources :threads do |map|
|
resources :threads do
|
||||||
map.resources :messages do |map|
|
resources :messages do
|
||||||
map.resources :comments
|
resources :comments
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
@ -433,10 +398,10 @@ def test_nested_restful_routes
|
|||||||
|
|
||||||
def test_nested_restful_routes_with_overwritten_defaults
|
def test_nested_restful_routes_with_overwritten_defaults
|
||||||
with_routing do |set|
|
with_routing do |set|
|
||||||
set.draw do |map|
|
set.draw do
|
||||||
map.resources :threads do |map|
|
resources :threads do
|
||||||
map.resources :messages, :name_prefix => nil do |map|
|
resources :messages, :name_prefix => nil do
|
||||||
map.resources :comments, :name_prefix => nil
|
resources :comments, :name_prefix => nil
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
@ -453,10 +418,10 @@ def test_nested_restful_routes_with_overwritten_defaults
|
|||||||
|
|
||||||
def test_shallow_nested_restful_routes
|
def test_shallow_nested_restful_routes
|
||||||
with_routing do |set|
|
with_routing do |set|
|
||||||
set.draw do |map|
|
set.draw do
|
||||||
map.resources :threads, :shallow => true do |map|
|
resources :threads, :shallow => true do
|
||||||
map.resources :messages do |map|
|
resources :messages do
|
||||||
map.resources :comments
|
resources :comments
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
@ -478,11 +443,11 @@ def test_shallow_nested_restful_routes
|
|||||||
|
|
||||||
def test_shallow_nested_restful_routes_with_namespaces
|
def test_shallow_nested_restful_routes_with_namespaces
|
||||||
with_routing do |set|
|
with_routing do |set|
|
||||||
set.draw do |map|
|
set.draw do
|
||||||
map.namespace :backoffice do |map|
|
namespace :backoffice do
|
||||||
map.namespace :admin do |map|
|
namespace :admin do
|
||||||
map.resources :products, :shallow => true do |map|
|
resources :products, :shallow => true do
|
||||||
map.resources :images
|
resources :images
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
@ -531,9 +496,9 @@ def test_should_create_multiple_singleton_resource_routes
|
|||||||
|
|
||||||
def test_should_create_nested_singleton_resource_routes
|
def test_should_create_nested_singleton_resource_routes
|
||||||
with_routing do |set|
|
with_routing do |set|
|
||||||
set.draw do |map|
|
set.draw do
|
||||||
map.resource :admin, :controller => 'admin' do |admin|
|
resource :admin, :controller => 'admin' do
|
||||||
admin.resource :account
|
resource :account
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
|
|
||||||
@ -544,8 +509,8 @@ def test_should_create_nested_singleton_resource_routes
|
|||||||
|
|
||||||
def test_resource_has_many_should_become_nested_resources
|
def test_resource_has_many_should_become_nested_resources
|
||||||
with_routing do |set|
|
with_routing do |set|
|
||||||
set.draw do |map|
|
set.draw do
|
||||||
map.resources :messages, :has_many => [ :comments, :authors ]
|
resources :messages, :has_many => [ :comments, :authors ]
|
||||||
end
|
end
|
||||||
|
|
||||||
assert_simply_restful_for :messages
|
assert_simply_restful_for :messages
|
||||||
@ -556,8 +521,8 @@ def test_resource_has_many_should_become_nested_resources
|
|||||||
|
|
||||||
def test_resources_has_many_hash_should_become_nested_resources
|
def test_resources_has_many_hash_should_become_nested_resources
|
||||||
with_routing do |set|
|
with_routing do |set|
|
||||||
set.draw do |map|
|
set.draw do
|
||||||
map.resources :threads, :has_many => { :messages => [ :comments, { :authors => :threads } ] }
|
resources :threads, :has_many => { :messages => [ :comments, { :authors => :threads } ] }
|
||||||
end
|
end
|
||||||
|
|
||||||
assert_simply_restful_for :threads
|
assert_simply_restful_for :threads
|
||||||
@ -570,8 +535,8 @@ def test_resources_has_many_hash_should_become_nested_resources
|
|||||||
|
|
||||||
def test_shallow_resource_has_many_should_become_shallow_nested_resources
|
def test_shallow_resource_has_many_should_become_shallow_nested_resources
|
||||||
with_routing do |set|
|
with_routing do |set|
|
||||||
set.draw do |map|
|
set.draw do
|
||||||
map.resources :messages, :has_many => [ :comments, :authors ], :shallow => true
|
resources :messages, :has_many => [ :comments, :authors ], :shallow => true
|
||||||
end
|
end
|
||||||
|
|
||||||
assert_simply_restful_for :messages, :shallow => true
|
assert_simply_restful_for :messages, :shallow => true
|
||||||
@ -582,8 +547,8 @@ def test_shallow_resource_has_many_should_become_shallow_nested_resources
|
|||||||
|
|
||||||
def test_resource_has_one_should_become_nested_resources
|
def test_resource_has_one_should_become_nested_resources
|
||||||
with_routing do |set|
|
with_routing do |set|
|
||||||
set.draw do |map|
|
set.draw do
|
||||||
map.resources :messages, :has_one => :logo
|
resources :messages, :has_one => :logo
|
||||||
end
|
end
|
||||||
|
|
||||||
assert_simply_restful_for :messages
|
assert_simply_restful_for :messages
|
||||||
@ -593,8 +558,8 @@ def test_resource_has_one_should_become_nested_resources
|
|||||||
|
|
||||||
def test_shallow_resource_has_one_should_become_shallow_nested_resources
|
def test_shallow_resource_has_one_should_become_shallow_nested_resources
|
||||||
with_routing do |set|
|
with_routing do |set|
|
||||||
set.draw do |map|
|
set.draw do
|
||||||
map.resources :messages, :has_one => :logo, :shallow => true
|
resources :messages, :has_one => :logo, :shallow => true
|
||||||
end
|
end
|
||||||
|
|
||||||
assert_simply_restful_for :messages, :shallow => true
|
assert_simply_restful_for :messages, :shallow => true
|
||||||
@ -638,9 +603,9 @@ def test_singleton_resource_with_two_member_actions_with_same_method
|
|||||||
|
|
||||||
def test_should_nest_resources_in_singleton_resource
|
def test_should_nest_resources_in_singleton_resource
|
||||||
with_routing do |set|
|
with_routing do |set|
|
||||||
set.draw do |map|
|
set.draw do
|
||||||
map.resource :account do |account|
|
resource :account do
|
||||||
account.resources :messages
|
resources :messages
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
|
|
||||||
@ -651,9 +616,9 @@ def test_should_nest_resources_in_singleton_resource
|
|||||||
|
|
||||||
def test_should_nest_resources_in_singleton_resource_with_path_prefix
|
def test_should_nest_resources_in_singleton_resource_with_path_prefix
|
||||||
with_routing do |set|
|
with_routing do |set|
|
||||||
set.draw do |map|
|
set.draw do
|
||||||
map.resource(:account, :path_prefix => ':site_id') do |account|
|
resource(:account, :path_prefix => ':site_id') do
|
||||||
account.resources :messages
|
resources :messages
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
|
|
||||||
@ -664,9 +629,9 @@ def test_should_nest_resources_in_singleton_resource_with_path_prefix
|
|||||||
|
|
||||||
def test_should_nest_singleton_resource_in_resources
|
def test_should_nest_singleton_resource_in_resources
|
||||||
with_routing do |set|
|
with_routing do |set|
|
||||||
set.draw do |map|
|
set.draw do
|
||||||
map.resources :threads do |thread|
|
resources :threads do
|
||||||
thread.resource :admin, :controller => 'admin'
|
resource :admin, :controller => 'admin'
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
|
|
||||||
@ -694,8 +659,8 @@ def test_should_not_allow_delete_or_put_on_collection_path
|
|||||||
def test_should_not_allow_invalid_head_method_for_member_routes
|
def test_should_not_allow_invalid_head_method_for_member_routes
|
||||||
with_routing do |set|
|
with_routing do |set|
|
||||||
assert_raise(ArgumentError) do
|
assert_raise(ArgumentError) do
|
||||||
set.draw do |map|
|
set.draw do
|
||||||
map.resources :messages, :member => {:something => :head}
|
resources :messages, :member => {:something => :head}
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
@ -704,8 +669,8 @@ def test_should_not_allow_invalid_head_method_for_member_routes
|
|||||||
def test_should_not_allow_invalid_http_methods_for_member_routes
|
def test_should_not_allow_invalid_http_methods_for_member_routes
|
||||||
with_routing do |set|
|
with_routing do |set|
|
||||||
assert_raise(ArgumentError) do
|
assert_raise(ArgumentError) do
|
||||||
set.draw do |map|
|
set.draw do
|
||||||
map.resources :messages, :member => {:something => :invalid}
|
resources :messages, :member => {:something => :invalid}
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
@ -713,9 +678,9 @@ def test_should_not_allow_invalid_http_methods_for_member_routes
|
|||||||
|
|
||||||
def test_resource_action_separator
|
def test_resource_action_separator
|
||||||
with_routing do |set|
|
with_routing do |set|
|
||||||
set.draw do |map|
|
set.draw do
|
||||||
map.resources :messages, :collection => {:search => :get}, :new => {:preview => :any}, :name_prefix => 'thread_', :path_prefix => '/threads/:thread_id'
|
resources :messages, :collection => {:search => :get}, :new => {:preview => :any}, :name_prefix => 'thread_', :path_prefix => '/threads/:thread_id'
|
||||||
map.resource :account, :member => {:login => :get}, :new => {:preview => :any}, :name_prefix => 'admin_', :path_prefix => '/admin'
|
resource :account, :member => {:login => :get}, :new => {:preview => :any}, :name_prefix => 'admin_', :path_prefix => '/admin'
|
||||||
end
|
end
|
||||||
|
|
||||||
action_separator = ActionController::Base.resource_action_separator
|
action_separator = ActionController::Base.resource_action_separator
|
||||||
@ -733,8 +698,8 @@ def test_resource_action_separator
|
|||||||
|
|
||||||
def test_new_style_named_routes_for_resource
|
def test_new_style_named_routes_for_resource
|
||||||
with_routing do |set|
|
with_routing do |set|
|
||||||
set.draw do |map|
|
set.draw do
|
||||||
map.resources :messages, :collection => {:search => :get}, :new => {:preview => :any}, :name_prefix => 'thread_', :path_prefix => '/threads/:thread_id'
|
resources :messages, :collection => {:search => :get}, :new => {:preview => :any}, :name_prefix => 'thread_', :path_prefix => '/threads/:thread_id'
|
||||||
end
|
end
|
||||||
assert_simply_restful_for :messages, :name_prefix => 'thread_', :path_prefix => 'threads/1/', :options => { :thread_id => '1' }
|
assert_simply_restful_for :messages, :name_prefix => 'thread_', :path_prefix => 'threads/1/', :options => { :thread_id => '1' }
|
||||||
assert_named_route "/threads/1/messages/search", "search_thread_messages_path", {}
|
assert_named_route "/threads/1/messages/search", "search_thread_messages_path", {}
|
||||||
@ -745,8 +710,8 @@ def test_new_style_named_routes_for_resource
|
|||||||
|
|
||||||
def test_new_style_named_routes_for_singleton_resource
|
def test_new_style_named_routes_for_singleton_resource
|
||||||
with_routing do |set|
|
with_routing do |set|
|
||||||
set.draw do |map|
|
set.draw do
|
||||||
map.resource :account, :member => {:login => :get}, :new => {:preview => :any}, :name_prefix => 'admin_', :path_prefix => '/admin'
|
resource :account, :member => {:login => :get}, :new => {:preview => :any}, :name_prefix => 'admin_', :path_prefix => '/admin'
|
||||||
end
|
end
|
||||||
assert_singleton_restful_for :account, :name_prefix => 'admin_', :path_prefix => 'admin/'
|
assert_singleton_restful_for :account, :name_prefix => 'admin_', :path_prefix => 'admin/'
|
||||||
assert_named_route "/admin/account/login", "login_admin_account_path", {}
|
assert_named_route "/admin/account/login", "login_admin_account_path", {}
|
||||||
@ -757,9 +722,9 @@ def test_new_style_named_routes_for_singleton_resource
|
|||||||
|
|
||||||
def test_resources_in_namespace
|
def test_resources_in_namespace
|
||||||
with_routing do |set|
|
with_routing do |set|
|
||||||
set.draw do |map|
|
set.draw do
|
||||||
map.namespace :backoffice do |backoffice|
|
namespace :backoffice do
|
||||||
backoffice.resources :products
|
resources :products
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
|
|
||||||
@ -769,9 +734,9 @@ def test_resources_in_namespace
|
|||||||
|
|
||||||
def test_resource_has_many_in_namespace
|
def test_resource_has_many_in_namespace
|
||||||
with_routing do |set|
|
with_routing do |set|
|
||||||
set.draw do |map|
|
set.draw do
|
||||||
map.namespace :backoffice do |backoffice|
|
namespace :backoffice do
|
||||||
backoffice.resources :products, :has_many => :tags
|
resources :products, :has_many => :tags
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
|
|
||||||
@ -782,9 +747,9 @@ def test_resource_has_many_in_namespace
|
|||||||
|
|
||||||
def test_resource_has_one_in_namespace
|
def test_resource_has_one_in_namespace
|
||||||
with_routing do |set|
|
with_routing do |set|
|
||||||
set.draw do |map|
|
set.draw do
|
||||||
map.namespace :backoffice do |backoffice|
|
namespace :backoffice do
|
||||||
backoffice.resources :products, :has_one => :manufacturer
|
resources :products, :has_one => :manufacturer
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
|
|
||||||
@ -795,10 +760,10 @@ def test_resource_has_one_in_namespace
|
|||||||
|
|
||||||
def test_resources_in_nested_namespace
|
def test_resources_in_nested_namespace
|
||||||
with_routing do |set|
|
with_routing do |set|
|
||||||
set.draw do |map|
|
set.draw do
|
||||||
map.namespace :backoffice do |backoffice|
|
namespace :backoffice do
|
||||||
backoffice.namespace :admin do |admin|
|
backoffice.namespace :admin do
|
||||||
admin.resources :products
|
resources :products
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
@ -809,8 +774,8 @@ def test_resources_in_nested_namespace
|
|||||||
|
|
||||||
def test_resources_using_namespace
|
def test_resources_using_namespace
|
||||||
with_routing do |set|
|
with_routing do |set|
|
||||||
set.draw do |map|
|
set.draw do
|
||||||
map.resources :products, :namespace => "backoffice/"
|
resources :products, :namespace => "backoffice/"
|
||||||
end
|
end
|
||||||
|
|
||||||
assert_simply_restful_for :products, :controller => "backoffice/products"
|
assert_simply_restful_for :products, :controller => "backoffice/products"
|
||||||
@ -819,9 +784,9 @@ def test_resources_using_namespace
|
|||||||
|
|
||||||
def test_nested_resources_using_namespace
|
def test_nested_resources_using_namespace
|
||||||
with_routing do |set|
|
with_routing do |set|
|
||||||
set.draw do |map|
|
set.draw do
|
||||||
map.namespace :backoffice do |backoffice|
|
namespace :backoffice do
|
||||||
backoffice.resources :products do |products|
|
resources :products do
|
||||||
products.resources :images
|
products.resources :images
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
@ -833,10 +798,10 @@ def test_nested_resources_using_namespace
|
|||||||
|
|
||||||
def test_nested_resources_in_nested_namespace
|
def test_nested_resources_in_nested_namespace
|
||||||
with_routing do |set|
|
with_routing do |set|
|
||||||
set.draw do |map|
|
set.draw do
|
||||||
map.namespace :backoffice do |backoffice|
|
namespace :backoffice do
|
||||||
backoffice.namespace :admin do |admin|
|
backoffice.namespace :admin do
|
||||||
admin.resources :products do |products|
|
resources :products do
|
||||||
products.resources :images
|
products.resources :images
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
@ -863,12 +828,12 @@ def test_with_path_segment
|
|||||||
|
|
||||||
def test_multiple_with_path_segment_and_controller
|
def test_multiple_with_path_segment_and_controller
|
||||||
with_routing do |set|
|
with_routing do |set|
|
||||||
set.draw do |map|
|
set.draw do
|
||||||
map.resources :products do |products|
|
resources :products do
|
||||||
products.resources :product_reviews, :as => 'reviews', :controller => 'messages'
|
products.resources :product_reviews, :as => 'reviews', :controller => 'messages'
|
||||||
end
|
end
|
||||||
map.resources :tutors do |tutors|
|
resources :tutors do
|
||||||
tutors.resources :tutor_reviews, :as => 'reviews', :controller => 'comments'
|
resources :tutor_reviews, :as => 'reviews', :controller => 'comments'
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
|
|
||||||
@ -877,17 +842,22 @@ def test_multiple_with_path_segment_and_controller
|
|||||||
end
|
end
|
||||||
end
|
end
|
||||||
|
|
||||||
def test_with_path_segment_path_prefix_requirements
|
def test_with_path_segment_path_prefix_constraints
|
||||||
expected_options = {:controller => 'messages', :action => 'show', :thread_id => '1.1.1', :id => '1'}
|
expected_options = {:controller => 'messages', :action => 'show', :thread_id => '1.1.1', :id => '1'}
|
||||||
with_restful_routing :messages, :as => 'comments',:path_prefix => '/thread/:thread_id', :requirements => { :thread_id => /[0-9]\.[0-9]\.[0-9]/ } do
|
with_routing do |set|
|
||||||
|
set.draw do
|
||||||
|
scope '/thread/:thread_id', :constraints => { :thread_id => /[0-9]\.[0-9]\.[0-9]/ } do
|
||||||
|
resources :messages, :as => 'comments'
|
||||||
|
end
|
||||||
|
end
|
||||||
assert_recognizes(expected_options, :path => 'thread/1.1.1/comments/1', :method => :get)
|
assert_recognizes(expected_options, :path => 'thread/1.1.1/comments/1', :method => :get)
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
|
|
||||||
def test_resource_has_only_show_action
|
def test_resource_has_only_show_action
|
||||||
with_routing do |set|
|
with_routing do |set|
|
||||||
set.draw do |map|
|
set.draw do
|
||||||
map.resources :products, :only => :show
|
resources :products, :only => :show
|
||||||
end
|
end
|
||||||
|
|
||||||
assert_resource_allowed_routes('products', {}, { :id => '1' }, :show, [:index, :new, :create, :edit, :update, :destroy])
|
assert_resource_allowed_routes('products', {}, { :id => '1' }, :show, [:index, :new, :create, :edit, :update, :destroy])
|
||||||
@ -897,8 +867,8 @@ def test_resource_has_only_show_action
|
|||||||
|
|
||||||
def test_singleton_resource_has_only_show_action
|
def test_singleton_resource_has_only_show_action
|
||||||
with_routing do |set|
|
with_routing do |set|
|
||||||
set.draw do |map|
|
set.draw do
|
||||||
map.resource :account, :only => :show
|
resource :account, :only => :show
|
||||||
end
|
end
|
||||||
|
|
||||||
assert_singleton_resource_allowed_routes('accounts', {}, :show, [:index, :new, :create, :edit, :update, :destroy])
|
assert_singleton_resource_allowed_routes('accounts', {}, :show, [:index, :new, :create, :edit, :update, :destroy])
|
||||||
@ -908,8 +878,8 @@ def test_singleton_resource_has_only_show_action
|
|||||||
|
|
||||||
def test_resource_does_not_have_destroy_action
|
def test_resource_does_not_have_destroy_action
|
||||||
with_routing do |set|
|
with_routing do |set|
|
||||||
set.draw do |map|
|
set.draw do
|
||||||
map.resources :products, :except => :destroy
|
resources :products, :except => :destroy
|
||||||
end
|
end
|
||||||
|
|
||||||
assert_resource_allowed_routes('products', {}, { :id => '1' }, [:index, :new, :create, :show, :edit, :update], :destroy)
|
assert_resource_allowed_routes('products', {}, { :id => '1' }, [:index, :new, :create, :show, :edit, :update], :destroy)
|
||||||
@ -919,8 +889,8 @@ def test_resource_does_not_have_destroy_action
|
|||||||
|
|
||||||
def test_singleton_resource_does_not_have_destroy_action
|
def test_singleton_resource_does_not_have_destroy_action
|
||||||
with_routing do |set|
|
with_routing do |set|
|
||||||
set.draw do |map|
|
set.draw do
|
||||||
map.resource :account, :except => :destroy
|
resource :account, :except => :destroy
|
||||||
end
|
end
|
||||||
|
|
||||||
assert_singleton_resource_allowed_routes('accounts', {}, [:new, :create, :show, :edit, :update], :destroy)
|
assert_singleton_resource_allowed_routes('accounts', {}, [:new, :create, :show, :edit, :update], :destroy)
|
||||||
@ -930,8 +900,8 @@ def test_singleton_resource_does_not_have_destroy_action
|
|||||||
|
|
||||||
def test_resource_has_only_create_action_and_named_route
|
def test_resource_has_only_create_action_and_named_route
|
||||||
with_routing do |set|
|
with_routing do |set|
|
||||||
set.draw do |map|
|
set.draw do
|
||||||
map.resources :products, :only => :create
|
resources :products, :only => :create
|
||||||
end
|
end
|
||||||
|
|
||||||
assert_resource_allowed_routes('products', {}, { :id => '1' }, :create, [:index, :new, :show, :edit, :update, :destroy])
|
assert_resource_allowed_routes('products', {}, { :id => '1' }, :create, [:index, :new, :show, :edit, :update, :destroy])
|
||||||
@ -943,8 +913,8 @@ def test_resource_has_only_create_action_and_named_route
|
|||||||
|
|
||||||
def test_resource_has_only_update_action_and_named_route
|
def test_resource_has_only_update_action_and_named_route
|
||||||
with_routing do |set|
|
with_routing do |set|
|
||||||
set.draw do |map|
|
set.draw do
|
||||||
map.resources :products, :only => :update
|
resources :products, :only => :update
|
||||||
end
|
end
|
||||||
|
|
||||||
assert_resource_allowed_routes('products', {}, { :id => '1' }, :update, [:index, :new, :create, :show, :edit, :destroy])
|
assert_resource_allowed_routes('products', {}, { :id => '1' }, :update, [:index, :new, :create, :show, :edit, :destroy])
|
||||||
@ -956,8 +926,8 @@ def test_resource_has_only_update_action_and_named_route
|
|||||||
|
|
||||||
def test_resource_has_only_destroy_action_and_named_route
|
def test_resource_has_only_destroy_action_and_named_route
|
||||||
with_routing do |set|
|
with_routing do |set|
|
||||||
set.draw do |map|
|
set.draw do
|
||||||
map.resources :products, :only => :destroy
|
resources :products, :only => :destroy
|
||||||
end
|
end
|
||||||
|
|
||||||
assert_resource_allowed_routes('products', {}, { :id => '1' }, :destroy, [:index, :new, :create, :show, :edit, :update])
|
assert_resource_allowed_routes('products', {}, { :id => '1' }, :destroy, [:index, :new, :create, :show, :edit, :update])
|
||||||
@ -969,8 +939,8 @@ def test_resource_has_only_destroy_action_and_named_route
|
|||||||
|
|
||||||
def test_singleton_resource_has_only_create_action_and_named_route
|
def test_singleton_resource_has_only_create_action_and_named_route
|
||||||
with_routing do |set|
|
with_routing do |set|
|
||||||
set.draw do |map|
|
set.draw do
|
||||||
map.resource :account, :only => :create
|
resource :account, :only => :create
|
||||||
end
|
end
|
||||||
|
|
||||||
assert_singleton_resource_allowed_routes('accounts', {}, :create, [:new, :show, :edit, :update, :destroy])
|
assert_singleton_resource_allowed_routes('accounts', {}, :create, [:new, :show, :edit, :update, :destroy])
|
||||||
@ -982,8 +952,8 @@ def test_singleton_resource_has_only_create_action_and_named_route
|
|||||||
|
|
||||||
def test_singleton_resource_has_only_update_action_and_named_route
|
def test_singleton_resource_has_only_update_action_and_named_route
|
||||||
with_routing do |set|
|
with_routing do |set|
|
||||||
set.draw do |map|
|
set.draw do
|
||||||
map.resource :account, :only => :update
|
resource :account, :only => :update
|
||||||
end
|
end
|
||||||
|
|
||||||
assert_singleton_resource_allowed_routes('accounts', {}, :update, [:new, :create, :show, :edit, :destroy])
|
assert_singleton_resource_allowed_routes('accounts', {}, :update, [:new, :create, :show, :edit, :destroy])
|
||||||
@ -995,8 +965,8 @@ def test_singleton_resource_has_only_update_action_and_named_route
|
|||||||
|
|
||||||
def test_singleton_resource_has_only_destroy_action_and_named_route
|
def test_singleton_resource_has_only_destroy_action_and_named_route
|
||||||
with_routing do |set|
|
with_routing do |set|
|
||||||
set.draw do |map|
|
set.draw do
|
||||||
map.resource :account, :only => :destroy
|
resource :account, :only => :destroy
|
||||||
end
|
end
|
||||||
|
|
||||||
assert_singleton_resource_allowed_routes('accounts', {}, :destroy, [:new, :create, :show, :edit, :update])
|
assert_singleton_resource_allowed_routes('accounts', {}, :destroy, [:new, :create, :show, :edit, :update])
|
||||||
@ -1008,8 +978,8 @@ def test_singleton_resource_has_only_destroy_action_and_named_route
|
|||||||
|
|
||||||
def test_resource_has_only_collection_action
|
def test_resource_has_only_collection_action
|
||||||
with_routing do |set|
|
with_routing do |set|
|
||||||
set.draw do |map|
|
set.draw do
|
||||||
map.resources :products, :except => :all, :collection => { :sale => :get }
|
resources :products, :except => :all, :collection => { :sale => :get }
|
||||||
end
|
end
|
||||||
|
|
||||||
assert_resource_allowed_routes('products', {}, { :id => '1' }, [], [:index, :new, :create, :show, :edit, :update, :destroy])
|
assert_resource_allowed_routes('products', {}, { :id => '1' }, [], [:index, :new, :create, :show, :edit, :update, :destroy])
|
||||||
@ -1022,8 +992,8 @@ def test_resource_has_only_collection_action
|
|||||||
|
|
||||||
def test_resource_has_only_member_action
|
def test_resource_has_only_member_action
|
||||||
with_routing do |set|
|
with_routing do |set|
|
||||||
set.draw do |map|
|
set.draw do
|
||||||
map.resources :products, :except => :all, :member => { :preview => :get }
|
resources :products, :except => :all, :member => { :preview => :get }
|
||||||
end
|
end
|
||||||
|
|
||||||
assert_resource_allowed_routes('products', {}, { :id => '1' }, [], [:index, :new, :create, :show, :edit, :update, :destroy])
|
assert_resource_allowed_routes('products', {}, { :id => '1' }, [], [:index, :new, :create, :show, :edit, :update, :destroy])
|
||||||
@ -1036,8 +1006,8 @@ def test_resource_has_only_member_action
|
|||||||
|
|
||||||
def test_singleton_resource_has_only_member_action
|
def test_singleton_resource_has_only_member_action
|
||||||
with_routing do |set|
|
with_routing do |set|
|
||||||
set.draw do |map|
|
set.draw do
|
||||||
map.resource :account, :except => :all, :member => { :signup => :get }
|
resource :account, :except => :all, :member => { :signup => :get }
|
||||||
end
|
end
|
||||||
|
|
||||||
assert_singleton_resource_allowed_routes('accounts', {}, [], [:new, :create, :show, :edit, :update, :destroy])
|
assert_singleton_resource_allowed_routes('accounts', {}, [], [:new, :create, :show, :edit, :update, :destroy])
|
||||||
@ -1050,9 +1020,9 @@ def test_singleton_resource_has_only_member_action
|
|||||||
|
|
||||||
def test_nested_resource_has_only_show_and_member_action
|
def test_nested_resource_has_only_show_and_member_action
|
||||||
with_routing do |set|
|
with_routing do |set|
|
||||||
set.draw do |map|
|
set.draw do
|
||||||
map.resources :products, :only => [:index, :show] do |product|
|
resources :products, :only => [:index, :show] do
|
||||||
product.resources :images, :member => { :thumbnail => :get }, :only => :show
|
resources :images, :member => { :thumbnail => :get }, :only => :show
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
|
|
||||||
@ -1066,9 +1036,9 @@ def test_nested_resource_has_only_show_and_member_action
|
|||||||
|
|
||||||
def test_nested_resource_does_not_inherit_only_option
|
def test_nested_resource_does_not_inherit_only_option
|
||||||
with_routing do |set|
|
with_routing do |set|
|
||||||
set.draw do |map|
|
set.draw do
|
||||||
map.resources :products, :only => :show do |product|
|
resources :products, :only => :show do
|
||||||
product.resources :images, :except => :destroy
|
resources :images, :except => :destroy
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
|
|
||||||
@ -1079,9 +1049,9 @@ def test_nested_resource_does_not_inherit_only_option
|
|||||||
|
|
||||||
def test_nested_resource_does_not_inherit_only_option_by_default
|
def test_nested_resource_does_not_inherit_only_option_by_default
|
||||||
with_routing do |set|
|
with_routing do |set|
|
||||||
set.draw do |map|
|
set.draw do
|
||||||
map.resources :products, :only => :show do |product|
|
resources :products, :only => :show do
|
||||||
product.resources :images
|
resources :images
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
|
|
||||||
@ -1092,9 +1062,9 @@ def test_nested_resource_does_not_inherit_only_option_by_default
|
|||||||
|
|
||||||
def test_nested_resource_does_not_inherit_except_option
|
def test_nested_resource_does_not_inherit_except_option
|
||||||
with_routing do |set|
|
with_routing do |set|
|
||||||
set.draw do |map|
|
set.draw do
|
||||||
map.resources :products, :except => :show do |product|
|
resources :products, :except => :show do
|
||||||
product.resources :images, :only => :destroy
|
resources :images, :only => :destroy
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
|
|
||||||
@ -1105,9 +1075,9 @@ def test_nested_resource_does_not_inherit_except_option
|
|||||||
|
|
||||||
def test_nested_resource_does_not_inherit_except_option_by_default
|
def test_nested_resource_does_not_inherit_except_option_by_default
|
||||||
with_routing do |set|
|
with_routing do |set|
|
||||||
set.draw do |map|
|
set.draw do
|
||||||
map.resources :products, :except => :show do |product|
|
resources :products, :except => :show do
|
||||||
product.resources :images
|
resources :images
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
|
|
||||||
@ -1118,8 +1088,8 @@ def test_nested_resource_does_not_inherit_except_option_by_default
|
|||||||
|
|
||||||
def test_default_singleton_restful_route_uses_get
|
def test_default_singleton_restful_route_uses_get
|
||||||
with_routing do |set|
|
with_routing do |set|
|
||||||
set.draw do |map|
|
set.draw do
|
||||||
map.resource :product
|
resource :product
|
||||||
end
|
end
|
||||||
|
|
||||||
assert_routing '/product', :controller => 'products', :action => 'show'
|
assert_routing '/product', :controller => 'products', :action => 'show'
|
||||||
@ -1135,15 +1105,41 @@ def test_singleton_resource_name_is_not_singularized
|
|||||||
|
|
||||||
protected
|
protected
|
||||||
def with_restful_routing(*args)
|
def with_restful_routing(*args)
|
||||||
|
options = args.extract_options!
|
||||||
|
collection_methods = options.delete(:collection)
|
||||||
|
member_methods = options.delete(:member)
|
||||||
|
path_prefix = options.delete(:path_prefix)
|
||||||
|
args.push(options)
|
||||||
|
|
||||||
with_routing do |set|
|
with_routing do |set|
|
||||||
set.draw { |map| map.resources(*args) }
|
set.draw do
|
||||||
|
scope(path_prefix || '') do
|
||||||
|
resources(*args) do
|
||||||
|
if collection_methods
|
||||||
|
collection do
|
||||||
|
collection_methods.each do |name, method|
|
||||||
|
send(method, name)
|
||||||
|
end
|
||||||
|
end
|
||||||
|
end
|
||||||
|
|
||||||
|
if member_methods
|
||||||
|
member do
|
||||||
|
member_methods.each do |name, method|
|
||||||
|
send(method, name)
|
||||||
|
end
|
||||||
|
end
|
||||||
|
end
|
||||||
|
end
|
||||||
|
end
|
||||||
|
end
|
||||||
yield
|
yield
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
|
|
||||||
def with_singleton_resources(*args)
|
def with_singleton_resources(*args)
|
||||||
with_routing do |set|
|
with_routing do |set|
|
||||||
set.draw { |map| map.resource(*args) }
|
set.draw {resource(*args) }
|
||||||
yield
|
yield
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
@ -1385,7 +1381,7 @@ def assert_not_recognizes(expected_options, path)
|
|||||||
end
|
end
|
||||||
|
|
||||||
def distinct_routes? (r1, r2)
|
def distinct_routes? (r1, r2)
|
||||||
if r1.conditions == r2.conditions and r1.requirements == r2.requirements then
|
if r1.conditions == r2.conditions and r1.constraints == r2.constraints then
|
||||||
if r1.segments.collect(&:to_s) == r2.segments.collect(&:to_s) then
|
if r1.segments.collect(&:to_s) == r2.segments.collect(&:to_s) then
|
||||||
return false
|
return false
|
||||||
end
|
end
|
||||||
|
File diff suppressed because it is too large
Load Diff
@ -135,6 +135,11 @@ def setup
|
|||||||
@request = ActionController::TestRequest.new
|
@request = ActionController::TestRequest.new
|
||||||
@response = ActionController::TestResponse.new
|
@response = ActionController::TestResponse.new
|
||||||
@request.env['PATH_INFO'] = nil
|
@request.env['PATH_INFO'] = nil
|
||||||
|
@routes = ActionDispatch::Routing::RouteSet.new.tap do |r|
|
||||||
|
r.draw do
|
||||||
|
match ':controller(/:action(/:id))'
|
||||||
|
end
|
||||||
|
end
|
||||||
end
|
end
|
||||||
|
|
||||||
def test_raw_post_handling
|
def test_raw_post_handling
|
||||||
@ -454,7 +459,7 @@ def test_assert_routing
|
|||||||
|
|
||||||
def test_assert_routing_with_method
|
def test_assert_routing_with_method
|
||||||
with_routing do |set|
|
with_routing do |set|
|
||||||
set.draw { |map| map.resources(:content) }
|
set.draw { resources(:content) }
|
||||||
assert_routing({ :method => 'post', :path => 'content' }, { :controller => 'content', :action => 'create' })
|
assert_routing({ :method => 'post', :path => 'content' }, { :controller => 'content', :action => 'create' })
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
@ -465,7 +470,7 @@ def test_assert_routing_in_module
|
|||||||
|
|
||||||
def test_assert_routing_with_glob
|
def test_assert_routing_with_glob
|
||||||
with_routing do |set|
|
with_routing do |set|
|
||||||
set.draw { |map| match('*path' => "pages#show") }
|
set.draw { match('*path' => "pages#show") }
|
||||||
assert_routing('/company/about', { :controller => 'pages', :action => 'show', :path => 'company/about' })
|
assert_routing('/company/about', { :controller => 'pages', :action => 'show', :path => 'company/about' })
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
@ -487,7 +492,7 @@ def test_id_converted_to_string
|
|||||||
|
|
||||||
def test_array_path_parameter_handled_properly
|
def test_array_path_parameter_handled_properly
|
||||||
with_routing do |set|
|
with_routing do |set|
|
||||||
set.draw do |map|
|
set.draw do
|
||||||
match 'file/*path', :to => 'test_test/test#test_params'
|
match 'file/*path', :to => 'test_test/test#test_params'
|
||||||
match ':controller/:action'
|
match ':controller/:action'
|
||||||
end
|
end
|
||||||
@ -702,7 +707,7 @@ class NamedRoutesControllerTest < ActionController::TestCase
|
|||||||
|
|
||||||
def test_should_be_able_to_use_named_routes_before_a_request_is_done
|
def test_should_be_able_to_use_named_routes_before_a_request_is_done
|
||||||
with_routing do |set|
|
with_routing do |set|
|
||||||
set.draw { |map| resources :contents }
|
set.draw { resources :contents }
|
||||||
assert_equal 'http://test.host/contents/new', new_content_url
|
assert_equal 'http://test.host/contents/new', new_content_url
|
||||||
assert_equal 'http://test.host/contents/1', content_url(:id => 1)
|
assert_equal 'http://test.host/contents/1', content_url(:id => 1)
|
||||||
end
|
end
|
||||||
|
@ -130,7 +130,7 @@ def test_relative_url_root_is_respected
|
|||||||
|
|
||||||
def test_named_routes
|
def test_named_routes
|
||||||
with_routing do |set|
|
with_routing do |set|
|
||||||
set.draw do |map|
|
set.draw do
|
||||||
match 'this/is/verbose', :to => 'home#index', :as => :no_args
|
match 'this/is/verbose', :to => 'home#index', :as => :no_args
|
||||||
match 'home/sweet/home/:user', :to => 'home#index', :as => :home
|
match 'home/sweet/home/:user', :to => 'home#index', :as => :home
|
||||||
end
|
end
|
||||||
@ -151,7 +151,7 @@ def test_named_routes
|
|||||||
|
|
||||||
def test_relative_url_root_is_respected_for_named_routes
|
def test_relative_url_root_is_respected_for_named_routes
|
||||||
with_routing do |set|
|
with_routing do |set|
|
||||||
set.draw do |map|
|
set.draw do
|
||||||
match '/home/sweet/home/:user', :to => 'home#index', :as => :home
|
match '/home/sweet/home/:user', :to => 'home#index', :as => :home
|
||||||
end
|
end
|
||||||
|
|
||||||
@ -165,7 +165,7 @@ def test_relative_url_root_is_respected_for_named_routes
|
|||||||
|
|
||||||
def test_only_path
|
def test_only_path
|
||||||
with_routing do |set|
|
with_routing do |set|
|
||||||
set.draw do |map|
|
set.draw do
|
||||||
match 'home/sweet/home/:user', :to => 'home#index', :as => :home
|
match 'home/sweet/home/:user', :to => 'home#index', :as => :home
|
||||||
match ':controller/:action/:id'
|
match ':controller/:action/:id'
|
||||||
end
|
end
|
||||||
@ -233,7 +233,7 @@ def test_path_generation_for_symbol_parameter_keys
|
|||||||
|
|
||||||
def test_named_routes_with_nil_keys
|
def test_named_routes_with_nil_keys
|
||||||
with_routing do |set|
|
with_routing do |set|
|
||||||
set.draw do |map|
|
set.draw do
|
||||||
match 'posts.:format', :to => 'posts#index', :as => :posts
|
match 'posts.:format', :to => 'posts#index', :as => :posts
|
||||||
match '/', :to => 'posts#index', :as => :main
|
match '/', :to => 'posts#index', :as => :main
|
||||||
end
|
end
|
||||||
|
@ -19,6 +19,11 @@ def setup
|
|||||||
@request = ActionController::TestRequest.new
|
@request = ActionController::TestRequest.new
|
||||||
@params = {}
|
@params = {}
|
||||||
@rewriter = Rewriter.new(@request) #.new(@request, @params)
|
@rewriter = Rewriter.new(@request) #.new(@request, @params)
|
||||||
|
@routes = ActionDispatch::Routing::RouteSet.new.tap do |r|
|
||||||
|
r.draw do
|
||||||
|
match ':controller(/:action(/:id))'
|
||||||
|
end
|
||||||
|
end
|
||||||
end
|
end
|
||||||
|
|
||||||
def test_port
|
def test_port
|
||||||
|
@ -254,7 +254,7 @@ def with_params_parsers(parsers = {})
|
|||||||
|
|
||||||
def with_test_route_set
|
def with_test_route_set
|
||||||
with_routing do |set|
|
with_routing do |set|
|
||||||
set.draw do |map|
|
set.draw do
|
||||||
match '/', :to => 'web_service_test/test#assign_parameters'
|
match '/', :to => 'web_service_test/test#assign_parameters'
|
||||||
end
|
end
|
||||||
yield
|
yield
|
||||||
|
@ -56,7 +56,7 @@ def assert_parses(expected, actual, headers = {})
|
|||||||
|
|
||||||
def with_test_routing
|
def with_test_routing
|
||||||
with_routing do |set|
|
with_routing do |set|
|
||||||
set.draw do |map|
|
set.draw do
|
||||||
match ':action', :to => ::JsonParamsParsingTest::TestController
|
match ':action', :to => ::JsonParamsParsingTest::TestController
|
||||||
end
|
end
|
||||||
yield
|
yield
|
||||||
|
@ -156,7 +156,7 @@ def parse_multipart(name)
|
|||||||
|
|
||||||
def with_test_routing
|
def with_test_routing
|
||||||
with_routing do |set|
|
with_routing do |set|
|
||||||
set.draw do |map|
|
set.draw do
|
||||||
match ':action', :to => 'multipart_params_parsing_test/test'
|
match ':action', :to => 'multipart_params_parsing_test/test'
|
||||||
end
|
end
|
||||||
yield
|
yield
|
||||||
|
@ -108,7 +108,7 @@ def teardown
|
|||||||
private
|
private
|
||||||
def assert_parses(expected, actual)
|
def assert_parses(expected, actual)
|
||||||
with_routing do |set|
|
with_routing do |set|
|
||||||
set.draw do |map|
|
set.draw do
|
||||||
match ':action', :to => ::QueryStringParsingTest::TestController
|
match ':action', :to => ::QueryStringParsingTest::TestController
|
||||||
end
|
end
|
||||||
|
|
||||||
|
@ -129,7 +129,7 @@ def teardown
|
|||||||
private
|
private
|
||||||
def with_test_routing
|
def with_test_routing
|
||||||
with_routing do |set|
|
with_routing do |set|
|
||||||
set.draw do |map|
|
set.draw do
|
||||||
match ':action', :to => ::UrlEncodedParamsParsingTest::TestController
|
match ':action', :to => ::UrlEncodedParamsParsingTest::TestController
|
||||||
end
|
end
|
||||||
yield
|
yield
|
||||||
|
@ -96,7 +96,7 @@ def call(env)
|
|||||||
private
|
private
|
||||||
def with_test_routing
|
def with_test_routing
|
||||||
with_routing do |set|
|
with_routing do |set|
|
||||||
set.draw do |map|
|
set.draw do
|
||||||
match ':action', :to => ::XmlParamsParsingTest::TestController
|
match ':action', :to => ::XmlParamsParsingTest::TestController
|
||||||
end
|
end
|
||||||
yield
|
yield
|
||||||
|
@ -298,7 +298,7 @@ def get(path, parameters = nil, env = {})
|
|||||||
|
|
||||||
def with_test_route_set(options = {})
|
def with_test_route_set(options = {})
|
||||||
with_routing do |set|
|
with_routing do |set|
|
||||||
set.draw do |map|
|
set.draw do
|
||||||
match ':action', :to => ::CookieStoreTest::TestController
|
match ':action', :to => ::CookieStoreTest::TestController
|
||||||
end
|
end
|
||||||
|
|
||||||
|
@ -174,7 +174,7 @@ def test_prevents_session_fixation
|
|||||||
private
|
private
|
||||||
def with_test_route_set
|
def with_test_route_set
|
||||||
with_routing do |set|
|
with_routing do |set|
|
||||||
set.draw do |map|
|
set.draw do
|
||||||
match ':action', :to => ::MemCacheStoreTest::TestController
|
match ':action', :to => ::MemCacheStoreTest::TestController
|
||||||
end
|
end
|
||||||
|
|
||||||
|
@ -314,7 +314,7 @@ def test_feed_xhtml
|
|||||||
private
|
private
|
||||||
def with_restful_routing(resources)
|
def with_restful_routing(resources)
|
||||||
with_routing do |set|
|
with_routing do |set|
|
||||||
set.draw do |map|
|
set.draw do
|
||||||
resources(resources)
|
resources(resources)
|
||||||
end
|
end
|
||||||
yield
|
yield
|
||||||
|
@ -172,7 +172,7 @@ class ATestHelperTest < ActionView::TestCase
|
|||||||
|
|
||||||
test "is able to use named routes" do
|
test "is able to use named routes" do
|
||||||
with_routing do |set|
|
with_routing do |set|
|
||||||
set.draw { |map| resources :contents }
|
set.draw { resources :contents }
|
||||||
assert_equal 'http://test.host/contents/new', new_content_url
|
assert_equal 'http://test.host/contents/new', new_content_url
|
||||||
assert_equal 'http://test.host/contents/1', content_url(:id => 1)
|
assert_equal 'http://test.host/contents/1', content_url(:id => 1)
|
||||||
end
|
end
|
||||||
@ -180,7 +180,7 @@ class ATestHelperTest < ActionView::TestCase
|
|||||||
|
|
||||||
test "named routes can be used from helper included in view" do
|
test "named routes can be used from helper included in view" do
|
||||||
with_routing do |set|
|
with_routing do |set|
|
||||||
set.draw { |map| resources :contents }
|
set.draw { resources :contents }
|
||||||
_helpers.module_eval do
|
_helpers.module_eval do
|
||||||
def render_from_helper
|
def render_from_helper
|
||||||
new_content_url
|
new_content_url
|
||||||
|
@ -47,7 +47,7 @@ def test_link_to_person
|
|||||||
private
|
private
|
||||||
def with_test_route_set
|
def with_test_route_set
|
||||||
with_routing do |set|
|
with_routing do |set|
|
||||||
set.draw do |map|
|
set.draw do
|
||||||
match 'people', :to => 'people#index', :as => :people
|
match 'people', :to => 'people#index', :as => :people
|
||||||
end
|
end
|
||||||
yield
|
yield
|
||||||
|
@ -394,7 +394,7 @@ def sort_query_string_params(uri)
|
|||||||
|
|
||||||
class UrlHelperControllerTest < ActionController::TestCase
|
class UrlHelperControllerTest < ActionController::TestCase
|
||||||
class UrlHelperController < ActionController::Base
|
class UrlHelperController < ActionController::Base
|
||||||
test_routes do |map|
|
test_routes do
|
||||||
match 'url_helper_controller_test/url_helper/show/:id',
|
match 'url_helper_controller_test/url_helper/show/:id',
|
||||||
:to => 'url_helper_controller_test/url_helper#show',
|
:to => 'url_helper_controller_test/url_helper#show',
|
||||||
:as => :show
|
:as => :show
|
||||||
@ -407,8 +407,7 @@ class UrlHelperController < ActionController::Base
|
|||||||
:to => 'url_helper_controller_test/url_helper#show_named_route',
|
:to => 'url_helper_controller_test/url_helper#show_named_route',
|
||||||
:as => :show_named_route
|
:as => :show_named_route
|
||||||
|
|
||||||
map.connect ":controller/:action/:id"
|
match "/:controller(/:action(/:id))"
|
||||||
# match "/:controller(/:action(/:id))"
|
|
||||||
|
|
||||||
match 'url_helper_controller_test/url_helper/normalize_recall_params',
|
match 'url_helper_controller_test/url_helper/normalize_recall_params',
|
||||||
:to => UrlHelperController.action(:normalize_recall),
|
:to => UrlHelperController.action(:normalize_recall),
|
||||||
|
Loading…
Reference in New Issue
Block a user