diff options
author | Jamis Buck <jamis@37signals.com> | 2006-06-07 16:27:14 +0000 |
---|---|---|
committer | Jamis Buck <jamis@37signals.com> | 2006-06-07 16:27:14 +0000 |
commit | 816d67b104f9ea59cd0ffaccbcc250b66b7905d0 (patch) | |
tree | 42f998e62d9cc71635898c06d4bf4a37880d95e9 /actionpack/test/controller | |
parent | ae7029ddd1fe11890479df119d0f5996c40c80de (diff) | |
download | rails-816d67b104f9ea59cd0ffaccbcc250b66b7905d0.tar.gz rails-816d67b104f9ea59cd0ffaccbcc250b66b7905d0.tar.bz2 rails-816d67b104f9ea59cd0ffaccbcc250b66b7905d0.zip |
Make sure passed routing options are not mutated by routing code. (closes #5314)
git-svn-id: http://svn-commit.rubyonrails.org/rails/trunk@4444 5ecf4fe2-1ee6-0310-87b1-e25e094e27de
Diffstat (limited to 'actionpack/test/controller')
-rw-r--r-- | actionpack/test/controller/routing_test.rb | 24 |
1 files changed, 21 insertions, 3 deletions
diff --git a/actionpack/test/controller/routing_test.rb b/actionpack/test/controller/routing_test.rb index 44e986e3d5..8838ba841e 100644 --- a/actionpack/test/controller/routing_test.rb +++ b/actionpack/test/controller/routing_test.rb @@ -863,11 +863,29 @@ class RouteTest < Test::Unit::TestCase end class RouteBuilderTest < Test::Unit::TestCase - + def builder - @bulider ||= ROUTING::RouteBuilder.new + @builder ||= ROUTING::RouteBuilder.new end - + + def build(path, options) + builder.build(path, options) + end + + def test_options_should_not_be_modified + requirements1 = { :id => /\w+/, :controller => /(?:[a-z](?:-?[a-z]+)*)/ } + requirements2 = requirements1.dup + + assert_equal requirements1, requirements2 + + with_options(:controller => 'folder', + :requirements => requirements2) do |m| + m.build 'folders/new', :action => 'new' + end + + assert_equal requirements1, requirements2 + end + def test_segment_for_static segment, rest = builder.segment_for 'ulysses' assert_equal '', rest |