Skip to content

Commit 10e3611

Browse files
committed
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.
1 parent e754787 commit 10e3611

File tree

1 file changed

+4
-1
lines changed

1 file changed

+4
-1
lines changed

activesupport/test/cache/stores/mem_cache_store_test.rb

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -85,14 +85,17 @@ def after_teardown
8585
def test_clear_also_clears_local_cache
8686
key = SecureRandom.uuid
8787
cache = lookup_store(raw: true)
88-
client.stub(:flush_all, -> { client.delete(key) }) do
88+
stub_called = false
89+
90+
client(cache).stub(:flush_all, -> { stub_called = true; client.delete("#{@namespace}:#{key}") }) do
8991
cache.with_local_cache do
9092
cache.write(key, SecureRandom.alphanumeric)
9193
cache.clear
9294
assert_nil cache.read(key)
9395
end
9496
assert_nil cache.read(key)
9597
end
98+
assert stub_called
9699
end
97100

98101
def test_raw_values

0 commit comments

Comments
 (0)