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/11/06 18:45:44 UTC
incubator-tinkerpop git commit: Fixed a minor issue in a test for
PartitionStrategy.
Repository: incubator-tinkerpop
Updated Branches:
refs/heads/master c2c58a749 -> 25be6cd31
Fixed a minor issue in a test for PartitionStrategy.
No real bug here except that all the asserts were not firing. Invoking CTR.
Project: http://git-wip-us.apache.org/repos/asf/incubator-tinkerpop/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-tinkerpop/commit/25be6cd3
Tree: http://git-wip-us.apache.org/repos/asf/incubator-tinkerpop/tree/25be6cd3
Diff: http://git-wip-us.apache.org/repos/asf/incubator-tinkerpop/diff/25be6cd3
Branch: refs/heads/master
Commit: 25be6cd31ea707d8d3aaf2a45362ef38a8bd5c6a
Parents: c2c58a7
Author: Stephen Mallette <sp...@genoprime.com>
Authored: Fri Nov 6 12:44:53 2015 -0500
Committer: Stephen Mallette <sp...@genoprime.com>
Committed: Fri Nov 6 12:44:53 2015 -0500
----------------------------------------------------------------------
.../decoration/PartitionStrategyProcessTest.java | 13 +++++++------
1 file changed, 7 insertions(+), 6 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/incubator-tinkerpop/blob/25be6cd3/gremlin-test/src/main/java/org/apache/tinkerpop/gremlin/process/traversal/strategy/decoration/PartitionStrategyProcessTest.java
----------------------------------------------------------------------
diff --git a/gremlin-test/src/main/java/org/apache/tinkerpop/gremlin/process/traversal/strategy/decoration/PartitionStrategyProcessTest.java b/gremlin-test/src/main/java/org/apache/tinkerpop/gremlin/process/traversal/strategy/decoration/PartitionStrategyProcessTest.java
index 10b887f..e4cea45 100644
--- a/gremlin-test/src/main/java/org/apache/tinkerpop/gremlin/process/traversal/strategy/decoration/PartitionStrategyProcessTest.java
+++ b/gremlin-test/src/main/java/org/apache/tinkerpop/gremlin/process/traversal/strategy/decoration/PartitionStrategyProcessTest.java
@@ -22,6 +22,7 @@ import org.apache.tinkerpop.gremlin.FeatureRequirement;
import org.apache.tinkerpop.gremlin.FeatureRequirementSet;
import org.apache.tinkerpop.gremlin.process.AbstractGremlinProcessTest;
import org.apache.tinkerpop.gremlin.process.traversal.dsl.graph.GraphTraversalSource;
+import org.apache.tinkerpop.gremlin.process.traversal.util.FastNoSuchElementException;
import org.apache.tinkerpop.gremlin.structure.Edge;
import org.apache.tinkerpop.gremlin.structure.Graph;
import org.apache.tinkerpop.gremlin.structure.Vertex;
@@ -340,11 +341,11 @@ public class PartitionStrategyProcessTest extends AbstractGremlinProcessTest {
assertEquals(vA.id(), sourceAA.V(vA.id()).id().next());
try {
- sourceA.V(vA.id());
+ sourceA.V(vA.id()).next();
+ fail("Vertex should not be in this partition");
} catch (Exception ex) {
- final Exception expected = Graph.Exceptions.elementNotFound(Vertex.class, vA.id());
+ final Exception expected = FastNoSuchElementException.instance();
assertEquals(expected.getClass(), ex.getClass());
- assertEquals(expected.getMessage(), ex.getMessage());
}
}
@@ -364,11 +365,11 @@ public class PartitionStrategyProcessTest extends AbstractGremlinProcessTest {
assertEquals(e.id(), g.E(e.id()).id().next());
try {
- sourceA.E(e.id());
+ sourceA.E(e.id()).next();
+ fail("Edge should not be in this partition");
} catch (Exception ex) {
- final Exception expected = Graph.Exceptions.elementNotFound(Edge.class, e.id());
+ final Exception expected = FastNoSuchElementException.instance();
assertEquals(expected.getClass(), ex.getClass());
- assertEquals(expected.getMessage(), ex.getMessage());
}
}