From 2b45e2d37093bd035085bddf0f3fa37192a25b5a Mon Sep 17 00:00:00 2001 From: Rick Olson Date: Sat, 2 Sep 2006 20:41:40 +0000 Subject: [PATCH] Add routing tests to assert that RoutingError is raised when conditions aren't met. Closes #6016 [Nathan Witmer] git-svn-id: http://svn-commit.rubyonrails.org/rails/trunk@4907 5ecf4fe2-1ee6-0310-87b1-e25e094e27de --- actionpack/CHANGELOG | 2 ++ actionpack/test/controller/routing_test.rb | 11 +++++++++++ 2 files changed, 13 insertions(+) diff --git a/actionpack/CHANGELOG b/actionpack/CHANGELOG index e8a729e8f5984..1dc0ebd65b087 100644 --- a/actionpack/CHANGELOG +++ b/actionpack/CHANGELOG @@ -1,5 +1,7 @@ *SVN* +* Add routing tests to assert that RoutingError is raised when conditions aren't met. Closes #6016 [Nathan Witmer] + * Deprecation: update docs. #5998 [jakob@mentalized.net, Kevin Clark] * Make auto_link parse a greater subset of valid url formats. [Jamis Buck] diff --git a/actionpack/test/controller/routing_test.rb b/actionpack/test/controller/routing_test.rb index f44614beb7f46..05d2a70f1c605 100644 --- a/actionpack/test/controller/routing_test.rb +++ b/actionpack/test/controller/routing_test.rb @@ -1381,6 +1381,13 @@ def test_recognize_with_conditions request.method = :post assert_nothing_raised { set.recognize(request) } assert_equal("create", request.path_parameters[:action]) + + request.method = :put + assert_nothing_raised { set.recognize(request) } + assert_equal("update", request.path_parameters[:action]) + + request.method = :update + assert_raises(ActionController::RoutingError) { set.recognize(request) } request.path = "/people/5" request.method = :get @@ -1397,6 +1404,10 @@ def test_recognize_with_conditions assert_nothing_raised { set.recognize(request) } assert_equal("destroy", request.path_parameters[:action]) assert_equal("5", request.path_parameters[:id]) + + request.method = :post + assert_raises(ActionController::RoutingError) { set.recognize(request) } + ensure Object.send(:remove_const, :PeopleController) end