aboutsummaryrefslogtreecommitdiffstats
path: root/actionpack/test
diff options
context:
space:
mode:
authorJeremy Daer <jeremydaer@gmail.com>2016-04-19 15:01:28 -0700
committerJeremy Daer <jeremydaer@gmail.com>2016-04-19 15:01:28 -0700
commitee25e1992c942cc3369b245a2736aff2a3bc5a02 (patch)
treec218bafe08bb712d7a5c458808bcbec23dad77bc /actionpack/test
parent0ff0add0400cdee5869d246315cc2551347187aa (diff)
parentfea7c9fed65b9dd305562483c1cfd80104d521b5 (diff)
downloadrails-ee25e1992c942cc3369b245a2736aff2a3bc5a02.tar.gz
rails-ee25e1992c942cc3369b245a2736aff2a3bc5a02.tar.bz2
rails-ee25e1992c942cc3369b245a2736aff2a3bc5a02.zip
Merge pull request #24031 from samphilipd/sam/do_not_clobber_options_in_route_definitions
Do not destructively mutate passed options hash in route definitions
Diffstat (limited to 'actionpack/test')
-rw-r--r--actionpack/test/dispatch/mapper_test.rb13
1 files changed, 13 insertions, 0 deletions
diff --git a/actionpack/test/dispatch/mapper_test.rb b/actionpack/test/dispatch/mapper_test.rb
index df27e41997..69098326b9 100644
--- a/actionpack/test/dispatch/mapper_test.rb
+++ b/actionpack/test/dispatch/mapper_test.rb
@@ -178,6 +178,19 @@ module ActionDispatch
mapper.mount as: "exciting"
end
end
+
+ def test_scope_does_not_destructively_mutate_default_options
+ fakeset = FakeSet.new
+ mapper = Mapper.new fakeset
+
+ frozen = { foo: :bar }.freeze
+
+ assert_nothing_raised do
+ mapper.scope(defaults: frozen) do
+ # pass
+ end
+ end
+ end
end
end
end