aboutsummaryrefslogtreecommitdiffstats
path: root/railties/test/rack_logger_test.rb
diff options
context:
space:
mode:
authorbogdanvlviv <bogdanvlviv@gmail.com>2018-04-06 13:50:15 +0300
committerbogdanvlviv <bogdanvlviv@gmail.com>2018-04-06 14:00:13 +0300
commitfd1c8c16c77c809098e85de960a7cde0a5025a68 (patch)
treeef03df6a4e6fd037c619f8051d9b77a7526f4ae9 /railties/test/rack_logger_test.rb
parentd514ce9199fac58e569482dc901e53d0526abdf7 (diff)
downloadrails-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/rack_logger_test.rb')
-rw-r--r--railties/test/rack_logger_test.rb6
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