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 2018/08/28 20:22:48 UTC
[1/2] tinkerpop git commit: Enabled some asserts that were commented
out on path serialization tests
Repository: tinkerpop
Updated Branches:
refs/heads/master e36256d6c -> ff2d7c0d4
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/master
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
[2/2] tinkerpop git commit: Merge branch 'tp33'
Posted by sp...@apache.org.
Merge branch 'tp33'
Project: http://git-wip-us.apache.org/repos/asf/tinkerpop/repo
Commit: http://git-wip-us.apache.org/repos/asf/tinkerpop/commit/ff2d7c0d
Tree: http://git-wip-us.apache.org/repos/asf/tinkerpop/tree/ff2d7c0d
Diff: http://git-wip-us.apache.org/repos/asf/tinkerpop/diff/ff2d7c0d
Branch: refs/heads/master
Commit: ff2d7c0d4b39ba6799b88fe7dd43bd224ae1a38b
Parents: e36256d 94d9de0
Author: Stephen Mallette <sp...@genoprime.com>
Authored: Tue Aug 28 16:22:13 2018 -0400
Committer: Stephen Mallette <sp...@genoprime.com>
Committed: Tue Aug 28 16:22:13 2018 -0400
----------------------------------------------------------------------
.../gremlin/structure/SerializationTest.java | 18 ++++++------------
1 file changed, 6 insertions(+), 12 deletions(-)
----------------------------------------------------------------------