Changeset 4907

Timestamp:

09/02/06 20:41:40 (2 years ago)

Author:

rick

Message:

Add routing tests to assert that RoutingError is raised when conditions aren't met. Closes #6016 [Nathan Witmer]

Files:

• trunk/actionpack/CHANGELOG (modified) (1 diff)
• trunk/actionpack/test/controller/routing_test.rb (modified) (2 diffs)

trunk/actionpack/CHANGELOG

@@ -1 +1 @@
-*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]

trunk/actionpack/test/controller/routing_test.rb

@@ -1382 +1382 @@
-    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"
@@ -1398 +1405 @@
-    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)