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/03/11 13:16:59 UTC

[tinkerpop] 21/26: 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 a7dd5f5136fa59b01489cfd2fb53d8943a780dc2
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 {