diff options
author | bogdanvlviv <bogdanvlviv@gmail.com> | 2018-04-06 13:50:15 +0300 |
---|---|---|
committer | bogdanvlviv <bogdanvlviv@gmail.com> | 2018-04-06 14:00:13 +0300 |
commit | fd1c8c16c77c809098e85de960a7cde0a5025a68 (patch) | |
tree | ef03df6a4e6fd037c619f8051d9b77a7526f4ae9 /railties/test | |
parent | d514ce9199fac58e569482dc901e53d0526abdf7 (diff) | |
download | rails-fd1c8c16c77c809098e85de960a7cde0a5025a68.tar.gz rails-fd1c8c16c77c809098e85de960a7cde0a5025a68.tar.bz2 rails-fd1c8c16c77c809098e85de960a7cde0a5025a68.zip |
Fix test added in #32444
Currently test `#test_logger_does_not_mutate_app_return` doesn't
test mutation of response and the test passes with and without changes
added in #32444. `#freeze` response in the test in order to
test mutation.
Diffstat (limited to 'railties/test')
-rw-r--r-- | railties/test/rack_logger_test.rb | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/railties/test/rack_logger_test.rb b/railties/test/rack_logger_test.rb index 41e063e893..6e8f333e1d 100644 --- a/railties/test/rack_logger_test.rb +++ b/railties/test/rack_logger_test.rb @@ -81,11 +81,13 @@ module Rails end def test_logger_does_not_mutate_app_return - response = [] + response = [].freeze app = TestApp.new(response) logger = TestLogger.new(app: app) assert_no_changes("response") do - logger.call("REQUEST_METHOD" => "GET") + assert_nothing_raised do + logger.call("REQUEST_METHOD" => "GET") + end end end end |