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

dcapwell pushed a commit to branch trunk
in repository https://gitbox.apache.org/repos/asf/cassandra-accord.git


The following commit(s) were added to refs/heads/trunk by this push:
     new 8b4f389  AccordGens.rangeDeps did not enforce unique ranges, which 
caused tests to fail
8b4f389 is described below

commit 8b4f3895cb926f937450676b1db2e23d01a8b820
Author: David Capwell <dcapw...@apache.org>
AuthorDate: Mon Apr 1 10:31:33 2024 -0700

    AccordGens.rangeDeps did not enforce unique ranges, which caused tests to 
fail
---
 accord-core/src/test/java/accord/utils/AccordGens.java | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/accord-core/src/test/java/accord/utils/AccordGens.java 
b/accord-core/src/test/java/accord/utils/AccordGens.java
index d87c2d5..971c70e 100644
--- a/accord-core/src/test/java/accord/utils/AccordGens.java
+++ b/accord-core/src/test/java/accord/utils/AccordGens.java
@@ -403,9 +403,10 @@ public class AccordGens
         return rs -> {
             if (rs.decide(emptyProb)) return RangeDeps.NONE;
             RangeDeps.Builder builder = RangeDeps.builder();
-            for (int i = 0, numKeys = rs.nextInt(1, 10); i < numKeys; i++)
+            List<? extends Range> uniqRanges = 
Gens.lists(rangeGen).uniqueBestEffort().ofSize(rs.nextInt(1, 10)).next(rs);
+            for (Range range : uniqRanges)
             {
-                builder.nextKey(rangeGen.next(rs));
+                builder.nextKey(range);
                 for (int j = 0, numTxn = rs.nextInt(1, 10); j < numTxn; j++)
                     builder.add(idGen.next(rs));
             }


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@cassandra.apache.org
For additional commands, e-mail: commits-h...@cassandra.apache.org

Reply via email to