Try to revert test changes

valkey-io · Apr 21, 2024 · c6496a7 · c6496a7
1 parent 47a8d78
commit c6496a7
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 7 deletions.
diff --git a/java/integTest/src/test/java/glide/SharedCommandTests.java b/java/integTest/src/test/java/glide/SharedCommandTests.java
@@ -1813,12 +1813,15 @@ public void brpop(BaseClient client) {
 
         // nothing popped out
         assertNull(
-                client.brpop(new String[] {listKey2}, REDIS_VERSION.isLowerThan("7.0.0") ? 1. : 0.5).get());
+                client
+                        .brpop(new String[] {listKey2}, REDIS_VERSION.isLowerThan("7.0.0") ? 1. : 0.0001)
+                        .get());
 
         // Key exists, but it is not a list
         assertEquals(OK, client.set("foo", "bar").get());
         ExecutionException executionException =
-                assertThrows(ExecutionException.class, () -> client.brpop(new String[] {"foo"}, 0.5).get());
+                assertThrows(
+                        ExecutionException.class, () -> client.brpop(new String[] {"foo"}, 0.0001).get());
         assertTrue(executionException.getCause() instanceof RequestException);
     }
 
@@ -1863,12 +1866,15 @@ public void blpop(BaseClient client) {
 
         // nothing popped out
         assertNull(
-                client.blpop(new String[] {listKey2}, REDIS_VERSION.isLowerThan("7.0.0") ? 1. : 0.5).get());
+                client
+                        .blpop(new String[] {listKey2}, REDIS_VERSION.isLowerThan("7.0.0") ? 1. : 0.0001)
+                        .get());
 
         // Key exists, but it is not a list
         assertEquals(OK, client.set("foo", "bar").get());
         ExecutionException executionException =
-                assertThrows(ExecutionException.class, () -> client.blpop(new String[] {"foo"}, 0.5).get());
+                assertThrows(
+                        ExecutionException.class, () -> client.blpop(new String[] {"foo"}, 0.0001).get());
         assertTrue(executionException.getCause() instanceof RequestException);
     }
 

diff --git a/node/tests/SharedTests.ts b/node/tests/SharedTests.ts
@@ -2041,14 +2041,14 @@ export function runBaseTests<Context>(config: {
                     await client.rpush("brpop-test", ["foo", "bar", "baz"]),
                 ).toEqual(3);
                 // Test basic usage
-                expect(await client.brpop(["brpop-test"], 0.5)).toEqual([
+                expect(await client.brpop(["brpop-test"], 0.1)).toEqual([
                     "brpop-test",
                     "baz",
                 ]);
                 // Delete all values from list
                 expect(await client.del(["brpop-test"])).toEqual(1);
-                // Test null return when key doesn't exist
-                expect(await client.brpop(["brpop-test"], 0.5)).toEqual(null);
+                // Test null return when key doesn't exis
+                expect(await client.brpop(["brpop-test"], 0.1)).toEqual(null);
             }, protocol);
         },
         config.timeout,