[tinkerpop] branch master updated: Fixed test assertions that were non-deterministic CTR

[spmallette]

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

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


The following commit(s) were added to refs/heads/master by this push:
     new 5ebf598  Fixed test assertions that were non-deterministic CTR
5ebf598 is described below

commit 5ebf598385badfeadb335b94c6139baac7601c34
Author: Stephen Mallette <sp...@genoprime.com>
AuthorDate: Thu Dec 20 06:59:31 2018 -0500

    Fixed test assertions that were non-deterministic CTR
---
 .../gremlin/process/traversal/step/branch/RepeatTest.java     | 11 ++++++++---
 1 file changed, 8 insertions(+), 3 deletions(-)

diff --git 
a/gremlin-test/src/main/java/org/apache/tinkerpop/gremlin/process/traversal/step/branch/RepeatTest.java
 
b/gremlin-test/src/main/java/org/apache/tinkerpop/gremlin/process/traversal/step/branch/RepeatTest.java
index dc4877e..c4175e5 100644
--- 
a/gremlin-test/src/main/java/org/apache/tinkerpop/gremlin/process/traversal/step/branch/RepeatTest.java
+++ 
b/gremlin-test/src/main/java/org/apache/tinkerpop/gremlin/process/traversal/step/branch/RepeatTest.java
@@ -341,7 +341,14 @@ public abstract class RepeatTest extends 
AbstractGremlinProcessTest {
     public void 
g_V_repeatXout_repeatXoutX_timesX1XX_timesX1X_limitX1X_path_by_name() {
         // This traversal gets optimised by the RepeatUnrollStrategy
         final Traversal<Vertex, Path> traversal_unrolled = 
get_g_V_repeatXout_repeatXoutX_timesX1XX_timesX1X_limitX1X_path_by_name();
-        final Path path_original = traversal_unrolled.next();
+        final Path pathOriginal = traversal_unrolled.next();
+        assertFalse(traversal_unrolled.hasNext());
+        assertEquals(3, pathOriginal.size());
+        assertEquals("marko", pathOriginal.get(0));
+        assertEquals("josh", pathOriginal.get(1));
+
+        // could be lop or ripple depending on what the graph chooses to 
traverse first
+        assertThat(pathOriginal.get(2), anyOf(equalTo("ripple"), 
equalTo("lop")));
 
         g = g.withoutStrategies(RepeatUnrollStrategy.class);
 
@@ -355,8 +362,6 @@ public abstract class RepeatTest extends 
AbstractGremlinProcessTest {
 
         // could be lop or ripple depending on what the graph chooses to 
traverse first
         assertThat(path.get(2), anyOf(equalTo("ripple"), equalTo("lop")));
-
-        assertEquals(path, path_original);
     }
 
     @Test