Minor: remove empty sequence checks

redis · Mar 19, 2022 · 9f76ec0 · 9f76ec0
1 parent 19585d4
commit 9f76ec0
Show file tree

Hide file tree

Showing 3 changed files with 2 additions and 9 deletions.
diff --git a/src/main/java/redis/clients/jedis/UnifiedJedis.java b/src/main/java/redis/clients/jedis/UnifiedJedis.java
@@ -3923,9 +3923,6 @@ public boolean bfExists(String key, String item) {
 
   @Override
   public List<Boolean> bfMExists(String key, String... items) {
-    if (items.length == 0) {
-      return Collections.emptyList();
-    }
     return executeCommand(commandObjects.bfMExists(key, items));
   }
 
@@ -3981,13 +3978,9 @@ public boolean cfExists(String key, String item) {
 
   @Override
   public List<Boolean> cfMExists(String key, String... items) {
-    if (items.length == 0) {
-      return Collections.emptyList();
-    }
     return executeCommand(commandObjects.cfMExists(key, items));
   }
 
-
   @Override
   public boolean cfDel(String key, String item) {
     return executeCommand(commandObjects.cfDel(key, item));

diff --git a/src/test/java/redis/clients/jedis/modules/bloom/BloomTest.java b/src/test/java/redis/clients/jedis/modules/bloom/BloomTest.java
@@ -217,7 +217,7 @@ public void insertExpansion() {
     assertEquals(20, insert.size());
   }
 
-  @Test
+  @Test(expected = JedisDataException.class)
   public void testNoItemMExists() {
     assertEquals(Collections.emptyList(), client.bfMExists("simpleBloom"));
   }

diff --git a/src/test/java/redis/clients/jedis/modules/bloom/CuckooTest.java b/src/test/java/redis/clients/jedis/modules/bloom/CuckooTest.java
@@ -191,7 +191,7 @@ public void testExistsItemExists() {
     assertEquals(Collections.singletonList(true), client.cfMExists("cuckoo17", "foo"));
   }
 
-  @Test
+  @Test(expected = JedisDataException.class)
   public void testNoItemMExists() {
     assertEquals(Collections.emptyList(), client.cfMExists("cuckoo17"));
   }