Permalink
Browse files

No longer need this special routing exception message because these k…

…inds of ambiguous generations are deprecated
  • Loading branch information...
1 parent 20f0b33 commit e00f57e20833a297efd1670890ebe5b030dbfdf1 @josh josh committed Oct 17, 2009
Showing with 0 additions and 13 deletions.
  1. +0 −13 actionpack/test/controller/routing_test.rb
@@ -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'

0 comments on commit e00f57e

Please sign in to comment.