Skip to content

Commit

Permalink
Drop legacy support for case insensitive controller recognition
Browse files Browse the repository at this point in the history
  • Loading branch information
josh committed Oct 18, 2009
1 parent cc0103f commit f1767c1
Showing 1 changed file with 0 additions and 11 deletions.
11 changes: 0 additions & 11 deletions actionpack/test/controller/routing_test.rb
Expand Up @@ -490,17 +490,6 @@ def test_action_expiry
assert_equal '/content', rs.generate({:controller => 'content'}, {:controller => 'content', :action => 'show'})
end

def test_recognition_with_uppercase_controller_name
@rs.draw {|m| m.connect ':controller/:action/:id' }
assert_equal({:controller => "content", :action => 'index'}, rs.recognize_path("/Content"))
assert_equal({:controller => "content", :action => 'list'}, rs.recognize_path("/ConTent/list"))
assert_equal({:controller => "content", :action => 'show', :id => '10'}, rs.recognize_path("/CONTENT/show/10"))

# these used to work, before the routes rewrite, but support for this was pulled in the new version...
#assert_equal({'controller' => "admin/news_feed", 'action' => 'index'}, rs.recognize_path("Admin/NewsFeed"))
#assert_equal({'controller' => "admin/news_feed", 'action' => 'index'}, rs.recognize_path("Admin/News_Feed"))
end

def test_requirement_should_prevent_optional_id
rs.draw do |map|
map.post 'post/:id', :controller=> 'post', :action=> 'show', :requirements => {:id => /\d+/}
Expand Down

0 comments on commit f1767c1

Please sign in to comment.