Changeset 8748

Timestamp:

01/30/08 01:25:44 (7 months ago)

Author:

nzkoz

Message:

Make assert_routing aware of the HTTP method used. Closes #8039 [mpalmer]

Files:

• trunk/actionpack/CHANGELOG (modified) (1 diff)
• trunk/actionpack/lib/action_controller/assertions/routing_assertions.rb (modified) (2 diffs)
• trunk/actionpack/test/controller/test_test.rb (modified) (1 diff)

trunk/actionpack/CHANGELOG

@@ -1 +1 @@
-*SVN*
+
+* Make assert_routing aware of the HTTP method used.  #8039 [mpalmer]
+  e.g. assert_routing({ :method => 'put', :path => '/product/321' }, { :controller => "product", :action => "update", :id => "321" })
-
-* Make map.root accept a single symbol as an argument to declare an alias.  #10818 [bscofield] 

trunk/actionpack/lib/action_controller/assertions/routing_assertions.rb

@@ -115 +115 @@
-      #  # Tests a route, providing a defaults hash
-      #  assert_routing 'controller/action/9', {:id => "9", :item => "square"}, {:controller => "controller", :action => "action"}, {}, {:item => "square"}
+      #
+      #  # Tests a route with a HTTP method
+      #  assert_routing({ :method => 'put', :path => '/product/321' }, { :controller => "product", :action => "update", :id => "321" })
-      def assert_routing(path, options, defaults={}, extras={}, message=nil)
-        assert_recognizes(options, path, extras, message)
@@ -123 +126 @@
-        end
-         
-
+.is_a?(Hash) ? path[:path] : path
-      end
-

trunk/actionpack/test/controller/test_test.rb

@@ -375 +375 @@
-  end
-
+  def test_assert_routing_with_method
+    with_routing do |set|
+        set.draw { |map| map.resources(:content) }
+      assert_routing({ :method => 'post', :path => 'content' }, { :controller => 'content', :action => 'create' })
+    end
+  end
+
-  def test_assert_routing_in_module
-    assert_routing 'admin/user', :controller => 'admin/user', :action => 'index'