Skip to content

Commit 2e9b4c7

Browse files
fix(jedis-compatibility): add missing closing braces in scripting methods
Fixed compilation errors caused by missing closing braces in: - functionStats() method in Jedis.java - testFunctionListMethodSignatures() method in JedisMethodsTest.java Also removed duplicate method declaration and leftover test code. Signed-off-by: prashanna-frsh <prashanna.rajendran@freshworks.com> Co-authored-by: Cursor <cursoragent@cursor.com>
1 parent 18b1e7f commit 2e9b4c7

File tree

2 files changed

+6
-6
lines changed

2 files changed

+6
-6
lines changed

java/jedis-compatibility/src/main/java/redis/clients/jedis/Jedis.java

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -7802,8 +7802,7 @@ public FunctionStats functionStats() {
78027802
return executeCommandWithGlide(
78037803
"FUNCTION STATS",
78047804
() -> {
7805-
Map<String, Map<String, Map<String, Object>>> result =
7806-
glideClient.functionStats().get();
7805+
Map<String, Map<String, Map<String, Object>>> result = glideClient.functionStats().get();
78077806
// The result structure is: { "running_script": {...}, "engines": {...} }
78087807
// But GLIDE returns it as Map<String, Map<String, Map<String, Object>>>
78097808
// We need to extract and flatten appropriately
@@ -7826,6 +7825,8 @@ public FunctionStats functionStats() {
78267825

78277826
return new FunctionStats(runningScript, engines);
78287827
});
7828+
}
7829+
78297830
/**
78307831
* Adds the specified members to the set stored at key.
78317832
*

java/jedis-compatibility/src/test/java/redis/clients/jedis/JedisMethodsTest.java

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -287,10 +287,9 @@ public void testFunctionListMethodSignatures() throws NoSuchMethodException {
287287
// Test functionStats() exists
288288
Method functionStats = jedisClass.getMethod("functionStats");
289289
assertEquals(FunctionStats.class, functionStats.getReturnType());
290-
Method aclDryRun =
291-
jedisClass.getMethod("aclDryRun", String.class, String.class, String[].class);
292-
assertEquals(String.class, aclDryRun.getReturnType());
293-
public void testSetMethodSignatures() throws NoSuchMethodException {
290+
}
291+
292+
@Test
294293
public void testAclMethodSignatures() throws NoSuchMethodException {
295294
Class<Jedis> jedisClass = Jedis.class;
296295

0 commit comments

Comments
 (0)