changed the SubgraphTest to use process API in assertions instead of structure 
API. This is in line with the current refactor trend to make sure the process 
API (and test suite) does not leak into the structure API. We need clear 
boundaries between the two APIs.


Project: http://git-wip-us.apache.org/repos/asf/tinkerpop/repo
Commit: http://git-wip-us.apache.org/repos/asf/tinkerpop/commit/797937eb
Tree: http://git-wip-us.apache.org/repos/asf/tinkerpop/tree/797937eb
Diff: http://git-wip-us.apache.org/repos/asf/tinkerpop/diff/797937eb

Branch: refs/heads/TINKERPOP-1489
Commit: 797937eb0c0b0fa3bc177925f4e584dfd53d8f13
Parents: f2b4fb9
Author: Marko A. Rodriguez <okramma...@gmail.com>
Authored: Thu Nov 2 10:07:10 2017 -0600
Committer: Marko A. Rodriguez <okramma...@gmail.com>
Committed: Thu Nov 2 10:07:10 2017 -0600

----------------------------------------------------------------------
 .../process/traversal/step/sideEffect/SubgraphTest.java | 12 ++++++------
 1 file changed, 6 insertions(+), 6 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/tinkerpop/blob/797937eb/gremlin-test/src/main/java/org/apache/tinkerpop/gremlin/process/traversal/step/sideEffect/SubgraphTest.java
----------------------------------------------------------------------
diff --git 
a/gremlin-test/src/main/java/org/apache/tinkerpop/gremlin/process/traversal/step/sideEffect/SubgraphTest.java
 
b/gremlin-test/src/main/java/org/apache/tinkerpop/gremlin/process/traversal/step/sideEffect/SubgraphTest.java
index 9f2a662..51d8870 100644
--- 
a/gremlin-test/src/main/java/org/apache/tinkerpop/gremlin/process/traversal/step/sideEffect/SubgraphTest.java
+++ 
b/gremlin-test/src/main/java/org/apache/tinkerpop/gremlin/process/traversal/step/sideEffect/SubgraphTest.java
@@ -71,15 +71,15 @@ public abstract class SubgraphTest extends 
AbstractGremlinProcessTest {
         assertVertexEdgeCounts(subgraph, 3, 2);
         subgraph.edges().forEachRemaining(e -> {
             assertEquals("knows", e.label());
-            assertEquals("marko", e.outVertex().values("name").next());
-            assertEquals(new Integer(29), 
e.outVertex().<Integer>values("age").next());
-            assertEquals("person", e.outVertex().label());
+            assertEquals("marko", g.E(e).outV().values("name").next());
+            assertEquals(new Integer(29), 
g.E(e).outV().<Integer>values("age").next());
+            assertEquals("person", g.E(e).outV().label().next());
 
-            final String name = e.inVertex().<String>values("name").next();
+            final String name = g.E(e).inV().<String>values("name").next();
             if (name.equals("vadas"))
-                assertEquals(0.5d, e.value("weight"), 0.0001d);
+                assertEquals(0.5d, g.E(e).<Double>values("weight").next(), 
0.0001d);
             else if (name.equals("josh"))
-                assertEquals(1.0d, e.value("weight"), 0.0001d);
+                assertEquals(1.0d, g.E(e).<Double>values("weight").next(), 
0.0001d);
             else
                 fail("There's a vertex present that should not be in the 
subgraph");
         });

Reply via email to