Fix MemCacheStoreTest stubbing

The stub wasn't called because it was defined on the `@cache` client
rather than on `cache`'s client.

This should reduce `MemCacheStoreTest` flakiness a bit.
This commit is contained in:
Jean Boussier 2022-02-16 13:03:25 +01:00
parent e7547876da
commit 10e361179d
1 changed files with 4 additions and 1 deletions

View File

@ -85,7 +85,9 @@ class MemCacheStoreTest < ActiveSupport::TestCase
def test_clear_also_clears_local_cache
key = SecureRandom.uuid
cache = lookup_store(raw: true)
client.stub(:flush_all, -> { client.delete(key) }) do
stub_called = false
client(cache).stub(:flush_all, -> { stub_called = true; client.delete("#{@namespace}:#{key}") }) do
cache.with_local_cache do
cache.write(key, SecureRandom.alphanumeric)
cache.clear
@ -93,6 +95,7 @@ class MemCacheStoreTest < ActiveSupport::TestCase
end
assert_nil cache.read(key)
end
assert stub_called
end
def test_raw_values