You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@geode.apache.org by ri...@apache.org on 2022/02/14 01:46:05 UTC

[geode] branch WIP-GEODE-9892 updated: LPUSH failing to update list length

This is an automated email from the ASF dual-hosted git repository.

ringles pushed a commit to branch WIP-GEODE-9892
in repository https://gitbox.apache.org/repos/asf/geode.git


The following commit(s) were added to refs/heads/WIP-GEODE-9892 by this push:
     new 1cea1ef  LPUSH failing to update list length
1cea1ef is described below

commit 1cea1ef4711d138dd46193c12f33ba42c3cc51e0
Author: Ray Ingles <ri...@vmware.com>
AuthorDate: Sun Feb 13 20:44:48 2022 -0500

    LPUSH failing to update list length
---
 .../geode/redis/internal/commands/executor/list/LPushDUnitTest.java    | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/geode-for-redis/src/distributedTest/java/org/apache/geode/redis/internal/commands/executor/list/LPushDUnitTest.java b/geode-for-redis/src/distributedTest/java/org/apache/geode/redis/internal/commands/executor/list/LPushDUnitTest.java
index c86b006..beb925b 100644
--- a/geode-for-redis/src/distributedTest/java/org/apache/geode/redis/internal/commands/executor/list/LPushDUnitTest.java
+++ b/geode-for-redis/src/distributedTest/java/org/apache/geode/redis/internal/commands/executor/list/LPushDUnitTest.java
@@ -128,7 +128,8 @@ public class LPushDUnitTest {
     for (int i = 0; i < MINIMUM_ITERATIONS; i++) {
       long listLength = jedis.llen(key);
       long newLength = jedis.lpush(key, elementList.toArray(new String[] {}));
-      assertThat(newLength - listLength).isGreaterThanOrEqualTo(3);
+      assertThat(newLength - listLength).as("LPUSH returned %s, LLEN was %s, diff of %s not >= 3",
+          newLength, listLength, newLength - listLength).isGreaterThanOrEqualTo(3);
       assertThat((newLength - listLength) % 3).isEqualTo(0);
     }
     runningCount.decrementAndGet();