From 2d78a3ae86f9f825f90a72de2693c567ecb5c4bd Mon Sep 17 00:00:00 2001 From: Santosh Wadghule Date: Thu, 14 Jul 2016 12:38:07 +0530 Subject: Add missing test for clearing up local cache on invalid parameters err. Add missing test for clearing up local cache and check response should be present on invalid parameters error. --- activesupport/test/caching_test.rb | 12 ++++++++++++ 1 file changed, 12 insertions(+) diff --git a/activesupport/test/caching_test.rb b/activesupport/test/caching_test.rb index 8002c14539..1e3810f6f7 100644 --- a/activesupport/test/caching_test.rb +++ b/activesupport/test/caching_test.rb @@ -25,6 +25,18 @@ module ActiveSupport assert_nil LocalCacheRegistry.cache_for(key) end + def test_local_cache_cleared_and_response_should_be_present_on_invalid_parameters_error + key = "super awesome key" + assert_nil LocalCacheRegistry.cache_for key + middleware = Middleware.new('<3', key).new(->(env) { + assert LocalCacheRegistry.cache_for(key), 'should have a cache' + raise Rack::Utils::InvalidParameterError + }) + response = middleware.call({}) + assert response, 'response should exist' + assert_nil LocalCacheRegistry.cache_for(key) + end + def test_local_cache_cleared_on_exception key = "super awesome key" assert_nil LocalCacheRegistry.cache_for key -- cgit v1.2.3