Enabled some asserts that were commented out on path serialization tests Not sure why this was commented out but it wasn't working at some point apparently - oh well, working now CTR
Project: http://git-wip-us.apache.org/repos/asf/tinkerpop/repo Commit: http://git-wip-us.apache.org/repos/asf/tinkerpop/commit/94d9de04 Tree: http://git-wip-us.apache.org/repos/asf/tinkerpop/tree/94d9de04 Diff: http://git-wip-us.apache.org/repos/asf/tinkerpop/diff/94d9de04 Branch: refs/heads/TINKERPOP-1774 Commit: 94d9de04c3d9ad22efd0cbd4b9d29a3dfb6e0b92 Parents: 5ca6998 Author: Stephen Mallette <sp...@genoprime.com> Authored: Tue Aug 28 16:20:57 2018 -0400 Committer: Stephen Mallette <sp...@genoprime.com> Committed: Tue Aug 28 16:20:57 2018 -0400 ---------------------------------------------------------------------- .../gremlin/structure/SerializationTest.java | 18 ++++++------------ 1 file changed, 6 insertions(+), 12 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/tinkerpop/blob/94d9de04/gremlin-test/src/main/java/org/apache/tinkerpop/gremlin/structure/SerializationTest.java ---------------------------------------------------------------------- diff --git a/gremlin-test/src/main/java/org/apache/tinkerpop/gremlin/structure/SerializationTest.java b/gremlin-test/src/main/java/org/apache/tinkerpop/gremlin/structure/SerializationTest.java index 4935175..0f910ab 100644 --- a/gremlin-test/src/main/java/org/apache/tinkerpop/gremlin/structure/SerializationTest.java +++ b/gremlin-test/src/main/java/org/apache/tinkerpop/gremlin/structure/SerializationTest.java @@ -722,27 +722,24 @@ public class SerializationTest { assertEquals(vOut.label(), detachedVOut.label()); assertEquals(vOut.id(), detachedVOut.id()); - // TODO: dunno GraphSON seems to return properties - will make this more consistent here // this is a SimpleTraverser so no properties are present in detachment - //assertFalse(detachedVOut.properties().hasNext()); + assertFalse(detachedVOut.properties().hasNext()); final Edge e = p.get("b"); final Edge detachedE = detached.get("b"); assertEquals(e.label(), detachedE.label()); assertEquals(e.id(), detachedE.id()); - // TODO: dunno GraphSON seems to return properties - will make this more consistent here // this is a SimpleTraverser so no properties are present in detachment - //assertFalse(detachedE.properties().hasNext()); + assertFalse(detachedE.properties().hasNext()); final Vertex vIn = p.get("c"); final Vertex detachedVIn = detached.get("c"); assertEquals(vIn.label(), detachedVIn.label()); assertEquals(vIn.id(), detachedVIn.id()); - // TODO: dunno GraphSON seems to return properties - will make this more consistent here // this is a SimpleTraverser so no properties are present in detachment - //assertFalse(detachedVIn.properties().hasNext()); + assertFalse(detachedVIn.properties().hasNext()); } @Test @@ -927,27 +924,24 @@ public class SerializationTest { assertEquals(vOut.label(), detachedVOut.label()); assertEquals(vOut.id(), detachedVOut.id()); - // TODO: dunno GraphSON seems to return properties - will make this more consistent here // this is a SimpleTraverser so no properties are present in detachment - //assertFalse(detachedVOut.properties().hasNext()); + assertFalse(detachedVOut.properties().hasNext()); final Edge e = p.get("b"); final Edge detachedE = detached.get("b"); assertEquals(e.label(), detachedE.label()); assertEquals(e.id(), detachedE.id()); - // TODO: dunno GraphSON seems to return properties - will make this more consistent here // this is a SimpleTraverser so no properties are present in detachment - //assertFalse(detachedE.properties().hasNext()); + assertFalse(detachedE.properties().hasNext()); final Vertex vIn = p.get("c"); final Vertex detachedVIn = detached.get("c"); assertEquals(vIn.label(), detachedVIn.label()); assertEquals(vIn.id(), detachedVIn.id()); - // TODO: dunno GraphSON seems to return properties - will make this more consistent here // this is a SimpleTraverser so no properties are present in detachment - //assertFalse(detachedVIn.properties().hasNext()); + assertFalse(detachedVIn.properties().hasNext()); } @Test
