Correct CommandArguments for zrevrangebyscore (#2972)

* Correct commandarguments for zrevrangebyscore * add unit test for zrevrangebyscore Co-authored-by: Daphne Hsieh <[email protected]>
redis · Apr 7, 2022 · a41c7c6 · a41c7c6
1 parent ec3582a
commit a41c7c6
Show file tree

Hide file tree

Showing 3 changed files with 15 additions and 1 deletion.
diff --git a/src/main/java/redis/clients/jedis/CommandObjects.java b/src/main/java/redis/clients/jedis/CommandObjects.java
@@ -1516,7 +1516,7 @@ public final CommandObject<List<String>> zrevrangeByScore(String key, double max
   }
 
   public final CommandObject<List<String>> zrevrangeByScore(String key, String max, String min, int offset, int count) {
-    return new CommandObject<>(commandArguments(ZRANGEBYSCORE).key(key).add(max).add(min)
+    return new CommandObject<>(commandArguments(ZREVRANGEBYSCORE).key(key).add(max).add(min)
         .add(LIMIT).add(offset).add(count), BuilderFactory.STRING_LIST);
   }
 

diff --git a/src/test/java/redis/clients/jedis/commands/jedis/SortedSetCommandsTest.java b/src/test/java/redis/clients/jedis/commands/jedis/SortedSetCommandsTest.java
@@ -859,6 +859,13 @@ public void zrevrangebyscore() {
 
     assertEquals(expected, range);
 
+    range = jedis.zrevrangeByScore("foo", "4", "2", 0, 2);
+    expected = new ArrayList<String>();
+    expected.add("d");
+    expected.add("c");
+
+    assertEquals(expected, range);
+
     range = jedis.zrevrangeByScore("foo", "+inf", "(4");
     expected = new ArrayList<String>();
     expected.add("e");

diff --git a/src/test/java/redis/clients/jedis/commands/unified/SortedSetCommandsTestBase.java b/src/test/java/redis/clients/jedis/commands/unified/SortedSetCommandsTestBase.java
@@ -916,6 +916,13 @@ public void zrevrangebyscore() {
 
     assertEquals(expected, range);
 
+    range = jedis.zrevrangeByScore("foo", "4", "2", 0, 2);
+    expected = new ArrayList<String>();
+    expected.add("d");
+    expected.add("c");
+
+    assertEquals(expected, range);
+
     range = jedis.zrevrangeByScore("foo", "+inf", "(4");
     expected = new ArrayList<String>();
     expected.add("e");