You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@tinkerpop.apache.org by ke...@apache.org on 2024/08/04 23:04:15 UTC

(tinkerpop) branch v4-io-test updated (14bc366f81 -> 7adc9d296f)

This is an automated email from the ASF dual-hosted git repository.

kenhuuu pushed a change to branch v4-io-test
in repository https://gitbox.apache.org/repos/asf/tinkerpop.git


    from 14bc366f81 Update tests to reflect v4 serialization types
     new 5d9b671a42 generators
     new 7adc9d296f added back some enums

The 2 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails.  The revisions
listed as "add" were already present in the repository and have only
been added to this reference.


Summary of changes:
 .../structure/graphbinary/GraphBinaryAccess.java   |   4 +-
 .../gremlin/structure/graphson/GraphSONAccess.java |   4 +-
 .../gremlin/structure/graphbinary/id-t-v4.gbin     | Bin 0 -> 10 bytes
 .../structure/graphbinary/neg-zero-double-v4.gbin  | Bin 0 -> 10 bytes
 .../structure/graphbinary/neg-zero-float-v4.gbin   | Bin 0 -> 6 bytes
 .../structure/graphbinary/out-direction-v4.gbin    | Bin
 .../io/AbstractTypedCompatibilityTest.java         |  51 ++++++++++++
 .../structure/io/GenerateGraphsonTestFiles.java    |  91 +++++++++++++++++++++
 .../gremlin/structure/io/GenerateTestFiles.java    |  78 ++++++++++++++++++
 .../tinkerpop/gremlin/structure/io/Model.java      |   5 ++
 .../graphson/GraphSONUntypedCompatibilityTest.java |  20 -----
 11 files changed, 231 insertions(+), 22 deletions(-)
 copy gremlin-console/src/test/python/x.script => gremlin-test/src/main/java/org/apache/tinkerpop/gremlin/structure/graphbinary/GraphBinaryAccess.java (89%)
 copy gremlin-console/src/test/python/x.script => gremlin-test/src/main/java/org/apache/tinkerpop/gremlin/structure/graphson/GraphSONAccess.java (90%)
 create mode 100644 gremlin-test/src/main/resources/org/apache/tinkerpop/gremlin/structure/graphbinary/id-t-v4.gbin
 create mode 100644 gremlin-test/src/main/resources/org/apache/tinkerpop/gremlin/structure/graphbinary/neg-zero-double-v4.gbin
 create mode 100644 gremlin-test/src/main/resources/org/apache/tinkerpop/gremlin/structure/graphbinary/neg-zero-float-v4.gbin
 copy gremlin-util/src/test/resources/org/apache/tinkerpop/gremlin/structure/io/graphbinary/direction-v1.gbin => gremlin-test/src/main/resources/org/apache/tinkerpop/gremlin/structure/graphbinary/out-direction-v4.gbin (100%)
 create mode 100644 gremlin-util/src/test/java/org/apache/tinkerpop/gremlin/structure/io/GenerateGraphsonTestFiles.java
 create mode 100644 gremlin-util/src/test/java/org/apache/tinkerpop/gremlin/structure/io/GenerateTestFiles.java


(tinkerpop) 02/02: added back some enums

Posted by ke...@apache.org.
This is an automated email from the ASF dual-hosted git repository.

kenhuuu pushed a commit to branch v4-io-test
in repository https://gitbox.apache.org/repos/asf/tinkerpop.git

commit 7adc9d296fe6dd6141b8ce3431e3d610f6f5af99
Author: Ken Hu <10...@users.noreply.github.com>
AuthorDate: Fri Aug 2 10:03:28 2024 -0700

    added back some enums
---
 .../gremlin/structure/graphbinary/id-t-v4.gbin     | Bin 0 -> 10 bytes
 .../structure/graphbinary/neg-zero-double-v4.gbin  | Bin 0 -> 10 bytes
 .../structure/graphbinary/neg-zero-float-v4.gbin   | Bin 0 -> 6 bytes
 .../structure/graphbinary/out-direction-v4.gbin    | Bin 0 -> 11 bytes
 .../io/AbstractTypedCompatibilityTest.java         |  51 +++++++++++++++++++++
 .../tinkerpop/gremlin/structure/io/Model.java      |   5 ++
 6 files changed, 56 insertions(+)

diff --git a/gremlin-test/src/main/resources/org/apache/tinkerpop/gremlin/structure/graphbinary/id-t-v4.gbin b/gremlin-test/src/main/resources/org/apache/tinkerpop/gremlin/structure/graphbinary/id-t-v4.gbin
new file mode 100644
index 0000000000..8a9235d39d
Binary files /dev/null and b/gremlin-test/src/main/resources/org/apache/tinkerpop/gremlin/structure/graphbinary/id-t-v4.gbin differ
diff --git a/gremlin-test/src/main/resources/org/apache/tinkerpop/gremlin/structure/graphbinary/neg-zero-double-v4.gbin b/gremlin-test/src/main/resources/org/apache/tinkerpop/gremlin/structure/graphbinary/neg-zero-double-v4.gbin
new file mode 100644
index 0000000000..f76a022bab
Binary files /dev/null and b/gremlin-test/src/main/resources/org/apache/tinkerpop/gremlin/structure/graphbinary/neg-zero-double-v4.gbin differ
diff --git a/gremlin-test/src/main/resources/org/apache/tinkerpop/gremlin/structure/graphbinary/neg-zero-float-v4.gbin b/gremlin-test/src/main/resources/org/apache/tinkerpop/gremlin/structure/graphbinary/neg-zero-float-v4.gbin
new file mode 100644
index 0000000000..cfd00ec653
Binary files /dev/null and b/gremlin-test/src/main/resources/org/apache/tinkerpop/gremlin/structure/graphbinary/neg-zero-float-v4.gbin differ
diff --git a/gremlin-test/src/main/resources/org/apache/tinkerpop/gremlin/structure/graphbinary/out-direction-v4.gbin b/gremlin-test/src/main/resources/org/apache/tinkerpop/gremlin/structure/graphbinary/out-direction-v4.gbin
new file mode 100644
index 0000000000..3caaba3c97
Binary files /dev/null and b/gremlin-test/src/main/resources/org/apache/tinkerpop/gremlin/structure/graphbinary/out-direction-v4.gbin differ
diff --git a/gremlin-util/src/test/java/org/apache/tinkerpop/gremlin/structure/io/AbstractTypedCompatibilityTest.java b/gremlin-util/src/test/java/org/apache/tinkerpop/gremlin/structure/io/AbstractTypedCompatibilityTest.java
index e881776185..027d63823f 100644
--- a/gremlin-util/src/test/java/org/apache/tinkerpop/gremlin/structure/io/AbstractTypedCompatibilityTest.java
+++ b/gremlin-util/src/test/java/org/apache/tinkerpop/gremlin/structure/io/AbstractTypedCompatibilityTest.java
@@ -288,6 +288,19 @@ public abstract class AbstractTypedCompatibilityTest extends AbstractCompatibili
         assertEquals(resource, recycled);
     }
 
+    @Test
+    public void shouldReadWriteNegativeZeroDouble() throws Exception {
+        final String resourceName = "neg-zero-double";
+
+        final Double resource = findModelEntryObject(resourceName);
+        final Double fromStatic = read(readFromResource(resourceName), Double.class);
+        final Double recycled = read(write(fromStatic, Double.class, resourceName), Double.class);
+        assertNotSame(fromStatic, recycled);
+        assertEquals(fromStatic, recycled);
+        assertEquals(resource, fromStatic);
+        assertEquals(resource, recycled);
+    }
+
     @Test
     public void shouldReadWriteZeroDuration() throws Exception {
         final String resourceName = "zero-duration";
@@ -434,6 +447,19 @@ public abstract class AbstractTypedCompatibilityTest extends AbstractCompatibili
         assertEquals(resource, recycled);
     }
 
+    @Test
+    public void shouldReadWriteNegativeZeroFloat() throws Exception {
+        final String resourceName = "neg-zero-float";
+
+        final Float resource = findModelEntryObject(resourceName);
+        final Float fromStatic = read(readFromResource(resourceName), Float.class);
+        final Float recycled = read(write(fromStatic, Float.class, resourceName), Float.class);
+        assertNotSame(fromStatic, recycled);
+        assertEquals(fromStatic, recycled);
+        assertEquals(resource, fromStatic);
+        assertEquals(resource, recycled);
+    }
+
     @Test
     public void shouldReadWriteMaxInteger() throws Exception {
         final String resourceName = "max-int";
@@ -828,4 +854,29 @@ public abstract class AbstractTypedCompatibilityTest extends AbstractCompatibili
         assertVertexProperty(resource, recycled);
         assertVertexProperty(resource, fromStatic);
     }
+
+    @Test
+    public void shouldReadWriteIdT() throws Exception {
+        final String resourceName = "id-t";
+
+        final T resource = findModelEntryObject(resourceName);
+        final T fromStatic = read(readFromResource(resourceName), T.class);
+        final T recycled = read(write(fromStatic, T.class, resourceName), T.class);
+        assertEquals(fromStatic, recycled);
+        assertEquals(resource, fromStatic);
+        assertEquals(resource, recycled);
+    }
+
+    @Test
+    public void shouldReadWriteOutDirection() throws Exception {
+        final String resourceName = "out-direction";
+
+        final Direction resource = findModelEntryObject(resourceName);
+        final Direction fromStatic = read(readFromResource(resourceName), Direction.class);
+        final Direction recycled = read(write(fromStatic, Direction.class, resourceName), Direction.class);
+        assertEquals(fromStatic, recycled);
+        assertEquals(resource, fromStatic);
+        assertEquals(resource, recycled);
+    }
+
 }
diff --git a/gremlin-util/src/test/java/org/apache/tinkerpop/gremlin/structure/io/Model.java b/gremlin-util/src/test/java/org/apache/tinkerpop/gremlin/structure/io/Model.java
index e2e1ee646d..7c2b081f9a 100644
--- a/gremlin-util/src/test/java/org/apache/tinkerpop/gremlin/structure/io/Model.java
+++ b/gremlin-util/src/test/java/org/apache/tinkerpop/gremlin/structure/io/Model.java
@@ -119,6 +119,7 @@ public class Model {
         addCoreEntry(Double.NaN, "nan-double");
         addCoreEntry(Double.POSITIVE_INFINITY, "pos-inf-double");
         addCoreEntry(Double.NEGATIVE_INFINITY, "neg-inf-double");
+        addCoreEntry(-0.0, "neg-zero-double");
 
         addCoreEntry(Float.MAX_VALUE, "max-float");
         addCoreEntry(Float.MIN_VALUE, "min-float");
@@ -127,6 +128,7 @@ public class Model {
         addCoreEntry(Float.NaN, "nan-float");
         addCoreEntry(Float.POSITIVE_INFINITY, "pos-inf-float");
         addCoreEntry(Float.NEGATIVE_INFINITY, "neg-inf-float");
+        addCoreEntry(-0.0f, "neg-zero-float");
 
         addCoreEntry(Integer.MAX_VALUE, "max-int");
         addCoreEntry(Integer.MIN_VALUE, "min-int");
@@ -205,6 +207,9 @@ public class Model {
 
         addExtendedEntry(Short.MIN_VALUE, "min-short", "");
         addExtendedEntry(Short.MAX_VALUE, "max-short", "");
+
+        addGraphProcessEntry(Direction.OUT, "out-direction", "");
+        addGraphProcessEntry(T.id, "id-t", "");
     }
 
     public static Model instance() {


(tinkerpop) 01/02: generators

Posted by ke...@apache.org.
This is an automated email from the ASF dual-hosted git repository.

kenhuuu pushed a commit to branch v4-io-test
in repository https://gitbox.apache.org/repos/asf/tinkerpop.git

commit 5d9b671a42010e00283e95c69ff961ca59ae3d2b
Author: Ken Hu <10...@users.noreply.github.com>
AuthorDate: Fri Jul 26 09:43:45 2024 -0700

    generators
---
 .../structure/graphbinary/GraphBinaryAccess.java   | 22 ++++++
 .../gremlin/structure/graphson/GraphSONAccess.java | 22 ++++++
 .../structure/io/GenerateGraphsonTestFiles.java    | 91 ++++++++++++++++++++++
 .../gremlin/structure/io/GenerateTestFiles.java    | 78 +++++++++++++++++++
 .../graphson/GraphSONUntypedCompatibilityTest.java | 20 -----
 5 files changed, 213 insertions(+), 20 deletions(-)

diff --git a/gremlin-test/src/main/java/org/apache/tinkerpop/gremlin/structure/graphbinary/GraphBinaryAccess.java b/gremlin-test/src/main/java/org/apache/tinkerpop/gremlin/structure/graphbinary/GraphBinaryAccess.java
new file mode 100644
index 0000000000..1eb973eab8
--- /dev/null
+++ b/gremlin-test/src/main/java/org/apache/tinkerpop/gremlin/structure/graphbinary/GraphBinaryAccess.java
@@ -0,0 +1,22 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contributor license agreements.  See the NOTICE file
+ * distributed with this work for additional information
+ * regarding copyright ownership.  The ASF licenses this file
+ * to you under the Apache License, Version 2.0 (the
+ * "License"); you may not use this file except in compliance
+ * with the License.  You may obtain a copy of the License at
+ *
+ * http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing,
+ * software distributed under the License is distributed on an
+ * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+ * KIND, either express or implied.  See the License for the
+ * specific language governing permissions and limitations
+ * under the License.
+ */
+package org.apache.tinkerpop.gremlin.structure.graphbinary;
+
+public class GraphBinaryAccess {
+}
diff --git a/gremlin-test/src/main/java/org/apache/tinkerpop/gremlin/structure/graphson/GraphSONAccess.java b/gremlin-test/src/main/java/org/apache/tinkerpop/gremlin/structure/graphson/GraphSONAccess.java
new file mode 100644
index 0000000000..7414abaa56
--- /dev/null
+++ b/gremlin-test/src/main/java/org/apache/tinkerpop/gremlin/structure/graphson/GraphSONAccess.java
@@ -0,0 +1,22 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contributor license agreements.  See the NOTICE file
+ * distributed with this work for additional information
+ * regarding copyright ownership.  The ASF licenses this file
+ * to you under the Apache License, Version 2.0 (the
+ * "License"); you may not use this file except in compliance
+ * with the License.  You may obtain a copy of the License at
+ *
+ * http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing,
+ * software distributed under the License is distributed on an
+ * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+ * KIND, either express or implied.  See the License for the
+ * specific language governing permissions and limitations
+ * under the License.
+ */
+package org.apache.tinkerpop.gremlin.structure.graphson;
+
+public class GraphSONAccess {
+}
diff --git a/gremlin-util/src/test/java/org/apache/tinkerpop/gremlin/structure/io/GenerateGraphsonTestFiles.java b/gremlin-util/src/test/java/org/apache/tinkerpop/gremlin/structure/io/GenerateGraphsonTestFiles.java
new file mode 100644
index 0000000000..d92bc14e6b
--- /dev/null
+++ b/gremlin-util/src/test/java/org/apache/tinkerpop/gremlin/structure/io/GenerateGraphsonTestFiles.java
@@ -0,0 +1,91 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contributor license agreements.  See the NOTICE file
+ * distributed with this work for additional information
+ * regarding copyright ownership.  The ASF licenses this file
+ * to you under the Apache License, Version 2.0 (the
+ * "License"); you may not use this file except in compliance
+ * with the License.  You may obtain a copy of the License at
+ *
+ * http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing,
+ * software distributed under the License is distributed on an
+ * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+ * KIND, either express or implied.  See the License for the
+ * specific language governing permissions and limitations
+ * under the License.
+ */
+package org.apache.tinkerpop.gremlin.structure.io;
+
+import io.netty.buffer.ByteBufAllocator;
+import org.apache.commons.configuration2.BaseConfiguration;
+import org.apache.commons.configuration2.Configuration;
+import org.apache.tinkerpop.gremlin.process.traversal.P;
+import org.apache.tinkerpop.gremlin.structure.Graph;
+import org.apache.tinkerpop.gremlin.structure.VertexProperty;
+import org.apache.tinkerpop.gremlin.structure.io.binary.GraphBinaryWriter;
+import org.apache.tinkerpop.gremlin.structure.io.graphson.GraphSONMapper;
+import org.apache.tinkerpop.gremlin.structure.io.graphson.GraphSONVersion;
+import org.apache.tinkerpop.gremlin.structure.io.graphson.GraphSONXModuleV3;
+import org.apache.tinkerpop.gremlin.structure.io.graphson.TypeInfo;
+import org.apache.tinkerpop.gremlin.tinkergraph.structure.TinkerFactory;
+import org.apache.tinkerpop.gremlin.tinkergraph.structure.TinkerGraph;
+import org.apache.tinkerpop.gremlin.tinkergraph.structure.TinkerIoRegistryV3;
+import org.apache.tinkerpop.gremlin.util.ser.AbstractGraphSONMessageSerializerV4;
+import org.apache.tinkerpop.gremlin.util.ser.NettyBufferFactory;
+import org.apache.tinkerpop.shaded.jackson.databind.ObjectMapper;
+
+import java.io.File;
+import java.io.FileOutputStream;
+import java.io.PrintWriter;
+
+/**
+ * Generates the test files that are associated with the Model. This class uses relative paths so it is intended to be
+ * run from the IDE and not from a JAR.
+ */
+public class GenerateGraphsonTestFiles {
+    private static ObjectMapper typedGraphSonMapper = GraphSONMapper.build().
+        addRegistry(TinkerIoRegistryV3.instance()).
+        addCustomModule(GraphSONXModuleV3.build()).
+        version(GraphSONVersion.V4_0).typeInfo(TypeInfo.PARTIAL_TYPES).create().createMapper();
+
+//    private static ObjectMapper untypedGraphSonMapper = GraphSONMapper.build().
+//            addRegistry(TinkerIoRegistryV3.instance()).
+//            typeInfo(TypeInfo.NO_TYPES).
+//            addCustomModule(GraphSONXModuleV3.build()).
+//            version(GraphSONVersion.V4_0).create().createMapper();
+    public static void main(String[] args) {
+        String basePath = "./test-case-data/io/graphson";
+        final File testDir = new File(basePath);
+
+        if (testDir.exists() || testDir.mkdirs()) {
+            Configuration conf = new BaseConfiguration();
+            conf.setProperty(TinkerGraph.GREMLIN_TINKERGRAPH_DEFAULT_VERTEX_PROPERTY_CARDINALITY, VertexProperty.Cardinality.list.name());
+            TinkerGraph graph = TinkerGraph.open(conf);
+            TinkerFactory.generateTheCrew(graph);
+
+            final Model model = Model.instance();
+
+            PrintWriter printer = null;
+            for (Model.Entry modelEntry : model.entries()) {
+                try {
+                    File fileToWriteTo = new File(basePath + "/" + modelEntry.getTitle().toLowerCase().replace(" ","") + "-" + "v4" + ".json");
+                    System.out.println("Writing " + modelEntry.getTitle());
+                    if (fileToWriteTo.exists()) fileToWriteTo.delete();
+                    printer = new PrintWriter(fileToWriteTo);
+
+                    String json = typedGraphSonMapper.writerWithDefaultPrettyPrinter().writeValueAsString(modelEntry.getObject());
+                    printer.print(json);
+
+                    printer.close();
+                } catch (Exception ex) {
+                    System.out.println("Exception occurred");
+                    ex.printStackTrace();
+                }
+            }
+        }
+    }
+
+
+}
\ No newline at end of file
diff --git a/gremlin-util/src/test/java/org/apache/tinkerpop/gremlin/structure/io/GenerateTestFiles.java b/gremlin-util/src/test/java/org/apache/tinkerpop/gremlin/structure/io/GenerateTestFiles.java
new file mode 100644
index 0000000000..b5f6d98c2d
--- /dev/null
+++ b/gremlin-util/src/test/java/org/apache/tinkerpop/gremlin/structure/io/GenerateTestFiles.java
@@ -0,0 +1,78 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contributor license agreements.  See the NOTICE file
+ * distributed with this work for additional information
+ * regarding copyright ownership.  The ASF licenses this file
+ * to you under the Apache License, Version 2.0 (the
+ * "License"); you may not use this file except in compliance
+ * with the License.  You may obtain a copy of the License at
+ *
+ * http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing,
+ * software distributed under the License is distributed on an
+ * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+ * KIND, either express or implied.  See the License for the
+ * specific language governing permissions and limitations
+ * under the License.
+ */
+package org.apache.tinkerpop.gremlin.structure.io;
+
+import io.netty.buffer.ByteBufAllocator;
+import org.apache.commons.configuration2.BaseConfiguration;
+import org.apache.commons.configuration2.Configuration;
+import org.apache.tinkerpop.gremlin.structure.Graph;
+import org.apache.tinkerpop.gremlin.structure.VertexProperty;
+import org.apache.tinkerpop.gremlin.structure.io.binary.GraphBinaryWriter;
+import org.apache.tinkerpop.gremlin.tinkergraph.structure.TinkerFactory;
+import org.apache.tinkerpop.gremlin.tinkergraph.structure.TinkerGraph;
+import org.apache.tinkerpop.gremlin.util.ser.NettyBufferFactory;
+
+import java.io.File;
+import java.io.FileOutputStream;
+
+/**
+ * Generates the test files that are associated with the Model. This class uses relative paths so it is intended to be
+ * run from the IDE and not from a JAR.
+ */
+public class GenerateTestFiles {
+    private static GraphBinaryWriter graphBinaryWriter = new GraphBinaryWriter();
+    public static void main(String[] args) {
+        String basePath = "./test-case-data/io/graphbinary";
+        final File testDir = new File(basePath);
+
+        if (testDir.exists() || testDir.mkdirs()) {
+            Configuration conf = new BaseConfiguration();
+            conf.setProperty(TinkerGraph.GREMLIN_TINKERGRAPH_DEFAULT_VERTEX_PROPERTY_CARDINALITY, VertexProperty.Cardinality.list.name());
+            TinkerGraph graph = TinkerGraph.open(conf);
+            TinkerFactory.generateTheCrew(graph);
+
+            final Model model = Model.instance();
+
+            ByteBufAllocator allocator = ByteBufAllocator.DEFAULT;
+            NettyBufferFactory factory = new NettyBufferFactory();
+            FileOutputStream filestream = null;
+            Buffer buffer = null;
+            for (Model.Entry modelEntry : model.entries()) {
+                try {
+                    File fileToWriteTo = new File(basePath + "/" + modelEntry.getTitle().toLowerCase().replace(" ","") + "-" + "v4" + ".gbin");
+                    System.out.println("Writing " + modelEntry.getTitle());
+                    if (fileToWriteTo.exists()) fileToWriteTo.delete();
+                    filestream = new FileOutputStream(fileToWriteTo);
+
+                    buffer = factory.create(allocator.buffer());
+                    graphBinaryWriter.write(modelEntry.getObject(), buffer);
+                    buffer.readerIndex(0);
+                    buffer.readBytes(filestream, buffer.readableBytes());
+                    filestream.close();
+                    buffer.release();
+                } catch (Exception ex) {
+                    System.out.println("Exception occurred");
+                    ex.printStackTrace();
+                }
+            }
+        }
+    }
+
+
+}
\ No newline at end of file
diff --git a/gremlin-util/src/test/java/org/apache/tinkerpop/gremlin/structure/io/graphson/GraphSONUntypedCompatibilityTest.java b/gremlin-util/src/test/java/org/apache/tinkerpop/gremlin/structure/io/graphson/GraphSONUntypedCompatibilityTest.java
index 48f2e741a1..82b8dfa648 100644
--- a/gremlin-util/src/test/java/org/apache/tinkerpop/gremlin/structure/io/graphson/GraphSONUntypedCompatibilityTest.java
+++ b/gremlin-util/src/test/java/org/apache/tinkerpop/gremlin/structure/io/graphson/GraphSONUntypedCompatibilityTest.java
@@ -39,23 +39,6 @@ import java.util.Arrays;
 @RunWith(Parameterized.class)
 public class GraphSONUntypedCompatibilityTest extends AbstractUntypedCompatibilityTest {
 
-    private static final ObjectMapper mapperV1 = GraphSONMapper.build().
-            addRegistry(TinkerIoRegistryV1.instance()).
-            typeInfo(TypeInfo.NO_TYPES).
-            version(GraphSONVersion.V1_0).create().createMapper();
-
-    private static final ObjectMapper mapperV2 = GraphSONMapper.build().
-                    addRegistry(TinkerIoRegistryV2.instance()).
-                    typeInfo(TypeInfo.NO_TYPES).
-                    addCustomModule(GraphSONXModuleV2.build()).
-                    version(GraphSONVersion.V2_0).create().createMapper();
-
-    private static final ObjectMapper mapperV3 = GraphSONMapper.build().
-            addRegistry(TinkerIoRegistryV3.instance()).
-            typeInfo(TypeInfo.NO_TYPES).
-            addCustomModule(GraphSONXModuleV3.build()).
-            version(GraphSONVersion.V3_0).create().createMapper();
-
     private static final ObjectMapper mapperV4 = GraphSONMapper.build().
             addRegistry(TinkerIoRegistryV3.instance()).
             typeInfo(TypeInfo.NO_TYPES).
@@ -73,9 +56,6 @@ public class GraphSONUntypedCompatibilityTest extends AbstractUntypedCompatibili
     @Parameterized.Parameters(name = "expect({0})")
     public static Iterable<Object[]> data() {
         return Arrays.asList(new Object[][]{
-                {"v1-no-types", mapperV1 },
-                {"v2-no-types", mapperV2 },
-                {"v3-no-types", mapperV3 },
                 {"v4-no-types", mapperV4 },
         });
     }