diff --git a/ContentGrabbingJi-CacheStore-redis/src/main/java/net/lamgc/cgj/bot/cache/redis/RedisMapCacheStore.java b/ContentGrabbingJi-CacheStore-redis/src/main/java/net/lamgc/cgj/bot/cache/redis/RedisMapCacheStore.java index e694b8f..6de0d2b 100644 --- a/ContentGrabbingJi-CacheStore-redis/src/main/java/net/lamgc/cgj/bot/cache/redis/RedisMapCacheStore.java +++ b/ContentGrabbingJi-CacheStore-redis/src/main/java/net/lamgc/cgj/bot/cache/redis/RedisMapCacheStore.java @@ -110,7 +110,7 @@ public class RedisMapCacheStore extends RedisCacheStore> imple public boolean putAll(CacheKey key, Map map) { Objects.requireNonNull(key); Objects.requireNonNull(map); - if (map.size() == 0 && exists(key)) { + if (map.size() == 0) { return true; } diff --git a/ContentGrabbingJi-CacheStore-redis/src/test/java/net/lamgc/cgj/bot/cache/redis/RedisMapCacheStoreTest.java b/ContentGrabbingJi-CacheStore-redis/src/test/java/net/lamgc/cgj/bot/cache/redis/RedisMapCacheStoreTest.java index 863c7ad..ef6c6f6 100644 --- a/ContentGrabbingJi-CacheStore-redis/src/test/java/net/lamgc/cgj/bot/cache/redis/RedisMapCacheStoreTest.java +++ b/ContentGrabbingJi-CacheStore-redis/src/test/java/net/lamgc/cgj/bot/cache/redis/RedisMapCacheStoreTest.java @@ -135,6 +135,9 @@ public class RedisMapCacheStoreTest { Assert.assertTrue("clearMap operation failed!", cacheStore.clearMap(key)); // putAll + // empty map + Assert.assertTrue(cacheStore.putAll(key, new HashMap<>())); + // non-empty map Assert.assertTrue(cacheStore.putAll(key, expectedMap)); Assert.assertTrue(expectedMap.keySet().containsAll(cacheStore.mapFieldSet(key))); Assert.assertTrue(expectedMap.values().containsAll(cacheStore.mapValueSet(key)));