diff options
author | David Verhasselt <david@crowdway.com> | 2019-04-12 15:40:29 +0300 |
---|---|---|
committer | David Verhasselt <david@crowdway.com> | 2019-04-12 16:12:14 +0300 |
commit | 5e21667496ded39e575640dcebe21cacd2e04ef6 (patch) | |
tree | cd6ab1dc0789491def2ad3cc1900b1390e5f4c1d /activesupport/test | |
parent | 3bd534287139549cee10e10364344d66139d9237 (diff) | |
download | rails-5e21667496ded39e575640dcebe21cacd2e04ef6.tar.gz rails-5e21667496ded39e575640dcebe21cacd2e04ef6.tar.bz2 rails-5e21667496ded39e575640dcebe21cacd2e04ef6.zip |
Redis fetch without names returns {}
When trying to call mget in Redis without any
parameters, a Redis error is thrown. To avoid
this, we circumvent Redis entirely when there
are no key names given.
Diffstat (limited to 'activesupport/test')
-rw-r--r-- | activesupport/test/cache/stores/redis_cache_store_test.rb | 6 |
1 files changed, 6 insertions, 0 deletions
diff --git a/activesupport/test/cache/stores/redis_cache_store_test.rb b/activesupport/test/cache/stores/redis_cache_store_test.rb index 1d87f74347..790534cd3c 100644 --- a/activesupport/test/cache/stores/redis_cache_store_test.rb +++ b/activesupport/test/cache/stores/redis_cache_store_test.rb @@ -140,6 +140,12 @@ module ActiveSupport::Cache::RedisCacheStoreTests end end + def test_fetch_multi_without_names + assert_not_called(@cache.redis, :mget) do + @cache.fetch_multi() { } + end + end + def test_increment_expires_in assert_called_with @cache.redis, :incrby, [ "#{@namespace}:foo", 1 ] do assert_called_with @cache.redis, :expire, [ "#{@namespace}:foo", 60 ] do |