fix to ';' interpretation in route recognition--only follow a route with ';' if the segment in question actually contains a ';'

git-svn-id: http://svn-commit.rubyonrails.org/rails/trunk@4244 5ecf4fe2-1ee6-0310-87b1-e25e094e27de

1 files changed, 17 insertions, 12 deletions

diff --git a/actionpack/test/controller/routing_test.rb b/actionpack/test/controller/routing_test.rb
index 4f12707f02..1562261735 100644
--- a/actionpack/test/controller/routing_test.rb
+++ b/actionpack/test/controller/routing_test.rb
@@ -1003,22 +1003,27 @@ class RouteSetTests < Test::Unit::TestCase
     Object.const_set(:SubpathBooksController, Class.new(ActionController::Base))
 
     rs.draw do |r|
-      r.connect '/books/:id;edit', :controller => 'subpath_books', :action => 'edit'
-      r.connect '/items/:id;:action', :controller => 'subpath_books'
+      #r.connect '/books/:id;edit', :controller => 'subpath_books', :action => 'edit'
+      #r.connect '/items/:id;:action', :controller => 'subpath_books'
       r.connect '/posts/new;:action', :controller => 'subpath_books'
+      r.connect '/posts/:id', :controller => 'subpath_books', :action => "show"
     end
 
-    hash = rs.recognize_path %w(books 17;edit)
-    assert_not_nil hash
-    assert_equal %w(subpath_books 17 edit), [hash["controller"].controller_name, hash["id"], hash["action"]]
-    
-    hash = rs.recognize_path %w(items 3;complete)
-    assert_not_nil hash
-    assert_equal %w(subpath_books 3 complete), [hash["controller"].controller_name, hash["id"], hash["action"]]
-    
-    hash = rs.recognize_path %w(posts new;preview)
+    # hash = rs.recognize_path %w(books 17;edit)
+    # assert_not_nil hash
+    # assert_equal %w(subpath_books 17 edit), [hash["controller"].controller_name, hash["id"], hash["action"]]
+   #  
+    # hash = rs.recognize_path %w(items 3;complete)
+    # assert_not_nil hash
+    # assert_equal %w(subpath_books 3 complete), [hash["controller"].controller_name, hash["id"], hash["action"]]
+   #  
+    # hash = rs.recognize_path %w(posts new;preview)
+    # assert_not_nil hash
+    # assert_equal %w(subpath_books preview), [hash["controller"].controller_name, hash["action"]]
+
+    hash = rs.recognize_path %w(posts 7)
     assert_not_nil hash
-    assert_equal %w(subpath_books preview), [hash["controller"].controller_name, hash["action"]]
+    assert_equal %w(subpath_books show 7), [hash["controller"].controller_name, hash["action"], hash["id"]]
 
     # for now, low-hanging fruit only. We don't allow subpath components anywhere
     # except at the end of the path