aboutsummaryrefslogtreecommitdiffstats
path: root/actionpack/test/dispatch/routing_test.rb
diff options
context:
space:
mode:
authorAndrew White <andyw@pixeltrix.co.uk>2013-01-21 17:20:51 +0000
committerAndrew White <andyw@pixeltrix.co.uk>2013-01-21 17:20:51 +0000
commit431759614d84395faa713ebdfa8bd1cadc97183a (patch)
treeedc38739164010569f5e45183ee47006352dadd7 /actionpack/test/dispatch/routing_test.rb
parenta3aca81b21f793e8869440e9e84ead80c2479e3d (diff)
downloadrails-431759614d84395faa713ebdfa8bd1cadc97183a.tar.gz
rails-431759614d84395faa713ebdfa8bd1cadc97183a.tar.bz2
rails-431759614d84395faa713ebdfa8bd1cadc97183a.zip
Duplicate possible frozen string from route
Ruby 1.9 freezes Hash string keys by default so where a route is defined like this: get 'search' => 'search' then the Mapper will derive the action from the key. This blows up later when the action is added to the parameters hash and the encoding is forced. Closes #3429
Diffstat (limited to 'actionpack/test/dispatch/routing_test.rb')
-rw-r--r--actionpack/test/dispatch/routing_test.rb7
1 files changed, 7 insertions, 0 deletions
diff --git a/actionpack/test/dispatch/routing_test.rb b/actionpack/test/dispatch/routing_test.rb
index 46d16598f7..6ecf011694 100644
--- a/actionpack/test/dispatch/routing_test.rb
+++ b/actionpack/test/dispatch/routing_test.rb
@@ -515,6 +515,8 @@ class TestRoutingMapper < ActionDispatch::IntegrationTest
match '/sculptors', :to => 'italians#sculptors'
match '/painters/:painter', :to => 'italians#painters', :constraints => {:painter => /michelangelo/}
end
+
+ get 'search' => 'search'
end
end
@@ -2477,6 +2479,11 @@ class TestRoutingMapper < ActionDispatch::IntegrationTest
assert_equal "/posts/1/admin", post_admin_root_path(:post_id => '1')
end
+ def test_action_from_path_is_not_frozen
+ get '/search'
+ assert !@request.params[:action].frozen?
+ end
+
private
def with_test_routes
yield