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 2015/03/26 19:54:37 UTC

[2/2] incubator-tinkerpop git commit: Removed a todo referring to more asserts on Subgraph testing.

Removed a todo referring to more asserts on Subgraph testing.

Will rely on tests in the gremlin-test suite for such things.  It is covered pretty well under there.


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

Branch: refs/heads/master
Commit: 9770afa63acfc2f60742f2349843237fb0294c52
Parents: bd16654
Author: Stephen Mallette <sp...@genoprime.com>
Authored: Thu Mar 26 14:52:52 2015 -0400
Committer: Stephen Mallette <sp...@genoprime.com>
Committed: Thu Mar 26 14:52:52 2015 -0400

----------------------------------------------------------------------
 .../strategy/decoration/SubgraphStrategyTraverseTest.java        | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/incubator-tinkerpop/blob/9770afa6/gremlin-core/src/test/java/org/apache/tinkerpop/gremlin/process/traversal/strategy/decoration/SubgraphStrategyTraverseTest.java
----------------------------------------------------------------------
diff --git a/gremlin-core/src/test/java/org/apache/tinkerpop/gremlin/process/traversal/strategy/decoration/SubgraphStrategyTraverseTest.java b/gremlin-core/src/test/java/org/apache/tinkerpop/gremlin/process/traversal/strategy/decoration/SubgraphStrategyTraverseTest.java
index cd7f146..3bfaa34 100644
--- a/gremlin-core/src/test/java/org/apache/tinkerpop/gremlin/process/traversal/strategy/decoration/SubgraphStrategyTraverseTest.java
+++ b/gremlin-core/src/test/java/org/apache/tinkerpop/gremlin/process/traversal/strategy/decoration/SubgraphStrategyTraverseTest.java
@@ -22,6 +22,7 @@ import org.apache.tinkerpop.gremlin.process.traversal.Traversal;
 import org.apache.tinkerpop.gremlin.process.traversal.dsl.graph.DefaultGraphTraversal;
 import org.apache.tinkerpop.gremlin.process.traversal.dsl.graph.__;
 import org.apache.tinkerpop.gremlin.process.traversal.step.filter.LambdaFilterStep;
+import org.apache.tinkerpop.gremlin.process.traversal.step.map.VertexStep;
 import org.apache.tinkerpop.gremlin.process.traversal.step.sideEffect.GraphStep;
 import org.apache.tinkerpop.gremlin.process.traversal.util.TraversalHelper;
 import org.apache.tinkerpop.gremlin.structure.Direction;
@@ -35,6 +36,7 @@ import java.util.Arrays;
 import java.util.List;
 
 import static org.junit.Assert.assertEquals;
+import static org.junit.Assert.assertTrue;
 import static org.mockito.Mockito.mock;
 
 /**
@@ -97,7 +99,5 @@ public class SubgraphStrategyTraverseTest {
 
         final List<LambdaFilterStep> steps = TraversalHelper.getStepsOfClass(LambdaFilterStep.class, traversal.asAdmin());
         assertEquals(expectedInsertedSteps, steps.size());
-
-        // todo: better assert expectations when exploding g.V().out() to g.V().outE().inV() and the like
     }
 }