diff options
author | Jeremy Daer <jeremydaer@gmail.com> | 2016-04-19 15:01:28 -0700 |
---|---|---|
committer | Jeremy Daer <jeremydaer@gmail.com> | 2016-04-19 15:01:28 -0700 |
commit | ee25e1992c942cc3369b245a2736aff2a3bc5a02 (patch) | |
tree | c218bafe08bb712d7a5c458808bcbec23dad77bc /actionpack/test | |
parent | 0ff0add0400cdee5869d246315cc2551347187aa (diff) | |
parent | fea7c9fed65b9dd305562483c1cfd80104d521b5 (diff) | |
download | rails-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.rb | 13 |
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 |