commit 8b4f3895cb926f937450676b1db2e23d01a8b820
author David Capwell <dcapwell@apache.org> Mon Apr 01 10:31:33 2024 -0700
committer David Capwell <dcapwell@apache.org> Mon Apr 01 10:31:33 2024 -0700
tree 25bd3204291a4f939ab859e43d141eb35c9294e8
parent 1a5cb4f10002fb3650ad464b3a77664f18e2a901

AccordGens.rangeDeps did not enforce unique ranges, which caused tests to fail

accord-core/src/test/java/accord/utils/AccordGens.java

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 @@
         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));
             }