You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@tinkerpop.apache.org by sp...@apache.org on 2020/04/06 12:41:52 UTC
[tinkerpop] 21/27: TINKERPOP-2076 Resolve test inconsistency
This is an automated email from the ASF dual-hosted git repository.
spmallette pushed a commit to branch TINKERPOP-2076
in repository https://gitbox.apache.org/repos/asf/tinkerpop.git
commit 42f621a2218ab60bdd76ac265077a7045c9ec9e0
Author: stephen <sp...@gmail.com>
AuthorDate: Wed Jan 8 10:24:15 2020 -0500
TINKERPOP-2076 Resolve test inconsistency
Missed a spot where we needed to explicitly count items in a ConcurrentSkipListSet rather than use size() which seems more sensitive in jdk11.
---
.../apache/tinkerpop/gremlin/process/computer/GraphComputerTest.java | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/gremlin-test/src/main/java/org/apache/tinkerpop/gremlin/process/computer/GraphComputerTest.java b/gremlin-test/src/main/java/org/apache/tinkerpop/gremlin/process/computer/GraphComputerTest.java
index a9ebd17..1cd5de5 100644
--- a/gremlin-test/src/main/java/org/apache/tinkerpop/gremlin/process/computer/GraphComputerTest.java
+++ b/gremlin-test/src/main/java/org/apache/tinkerpop/gremlin/process/computer/GraphComputerTest.java
@@ -1288,7 +1288,7 @@ public class GraphComputerTest extends AbstractGremlinProcessTest {
@Override
public void reduce(final NullObject key, final Iterator<Integer> values, final ReduceEmitter<NullObject, Integer> emitter) {
emitter.emit(3);
- if (WORKER_START.size() == 2) {
+ if (IteratorUtils.count(WORKER_START) == 2) {
assertEquals(2, countTrue(WORKER_START));
assertTrue(WORKER_START.contains(Stage.MAP) && WORKER_START.contains(Stage.REDUCE));
} else {