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 2019/06/17 21:25:38 UTC

[tinkerpop] 03/03: TINKERPOP-2238 Fixed GraphComputerTest iterator leak

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

spmallette pushed a commit to branch TINKERPOP-2238
in repository https://gitbox.apache.org/repos/asf/tinkerpop.git

commit d89bb7861008245a9608967df9d018c54efcf062
Author: Stephen Mallette <sp...@genoprime.com>
AuthorDate: Mon Jun 17 17:24:27 2019 -0400

    TINKERPOP-2238 Fixed GraphComputerTest iterator leak
---
 .../tinkerpop/gremlin/process/computer/GraphComputerTest.java      | 7 ++++++-
 1 file changed, 6 insertions(+), 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 b277894..4d4d4d2 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
@@ -47,6 +47,7 @@ import org.apache.tinkerpop.gremlin.structure.Graph;
 import org.apache.tinkerpop.gremlin.structure.Property;
 import org.apache.tinkerpop.gremlin.structure.Vertex;
 import org.apache.tinkerpop.gremlin.structure.VertexProperty;
+import org.apache.tinkerpop.gremlin.structure.util.CloseableIterator;
 import org.apache.tinkerpop.gremlin.structure.util.StringFactory;
 import org.apache.tinkerpop.gremlin.util.iterator.IteratorUtils;
 import org.javatuples.Pair;
@@ -2224,7 +2225,11 @@ public class GraphComputerTest extends AbstractGremlinProcessTest {
         assertEquals(6l, result.graph().traversal().V().properties("v3").count().next().longValue());
         assertEquals(6l, result.graph().traversal().V().<String>values("name").dedup().count().next().longValue());
         assertEquals(1l, result.graph().traversal().V().<String>values("v3").dedup().count().next().longValue());
-        assertEquals("shouldExist", result.graph().traversal().V().<String>values("v3").dedup().next());
+
+        final Traversal<Vertex,String> t = result.graph().traversal().V().<String>values("v3").dedup();
+        assertEquals("shouldExist", t.next());
+        CloseableIterator.closeIterator(t);
+
         ///
         assertFalse(result.memory().exists("m1"));
         assertFalse(result.memory().exists("m2"));