Permalink
Browse files

This is routing error message test is tightly coupled to the implemen…

…tation. Just test that it raises an exception.
  • Loading branch information...
1 parent f1767c1 commit 2c3ca9ae80c0ec30ce1aede77c4dafc55bf2957e @josh josh committed Oct 18, 2009
Showing with 1 addition and 16 deletions.
  1. +1 −16 actionpack/test/controller/routing_test.rb
@@ -407,22 +407,7 @@ def test_should_list_options_diff_when_routing_requirements_dont_match
rs.draw do |map|
map.post 'post/:id', :controller=> 'post', :action=> 'show', :requirements => {:id => /\d+/}
end
- exception = assert_raise(ActionController::RoutingError) { rs.generate(:controller => 'post', :action => 'show', :bad_param => "foo", :use_route => "post") }
- assert_match /^post_url failed to generate/, exception.message
- from_match = exception.message.match(/from \{[^\}]+\}/).to_s
- assert_match /:bad_param=>"foo"/, from_match
- assert_match /:action=>"show"/, from_match
- assert_match /:controller=>"post"/, from_match
-
- expected_match = exception.message.match(/expected: \{[^\}]+\}/).to_s
- assert_no_match /:bad_param=>"foo"/, expected_match
- assert_match /:action=>"show"/, expected_match
- assert_match /:controller=>"post"/, expected_match
-
- diff_match = exception.message.match(/diff: \{[^\}]+\}/).to_s
- assert_match /:bad_param=>"foo"/, diff_match
- assert_no_match /:action=>"show"/, diff_match
- assert_no_match /:controller=>"post"/, diff_match
+ assert_raise(ActionController::RoutingError) { rs.generate(:controller => 'post', :action => 'show', :bad_param => "foo", :use_route => "post") }
end
def test_dynamic_path_allowed

0 comments on commit 2c3ca9a

Please sign in to comment.