TEST_CODEOWNERS/rails
3
0
Fork 0
Code Issues Pull Requests 1 Actions Packages Projects Releases Wiki Activity

No longer need this special routing exception message because these kinds of ambiguous generations are deprecated

Browse Source
This commit is contained in:
Joshua Peek 2009-10-17 16:43:17 -05:00
parent 20f0b33035
commit e00f57e208
1 changed files with 0 additions and 13 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

13
actionpack/test/controller/routing_test.rb
View File

@ -425,19 +425,6 @@ def test_should_list_options_diff_when_routing_requirements_dont_match
assert_no_match /:controller=>"post"/, diff_match
end
# this specifies the case where your formerly would get a very confusing error message with an empty diff
def test_should_have_better_error_message_when_options_diff_is_empty
rs.draw do |map|
map.content '/content/:query', :controller => 'content', :action => 'show'
end
exception = assert_raise(ActionController::RoutingError) { rs.generate(:controller => 'content', :action => 'show', :use_route => "content") }
assert_match %r[:action=>"show"], exception.message
assert_match %r[:controller=>"content"], exception.message
assert_match %r[you may have ambiguous routes, or you may need to supply additional parameters for this route], exception.message
assert_match %r[content_url has the following required parameters: \["content", :query\] - are they all satisfied?], exception.message
end
def test_dynamic_path_allowed
rs.draw do |map|
map.connect '*path', :controller => 'content', :action => 'show_file'