Github user spmallette commented on a diff in the pull request:

    https://github.com/apache/tinkerpop/pull/934#discussion_r219606770
  
    --- Diff: 
gremlin-groovy-test/src/main/java/org/apache/tinkerpop/gremlin/groovy/jsr223/GroovyTranslatorTest.java
 ---
    @@ -141,4 +144,52 @@ public void shouldHandleEmptyMaps() {
         public void shouldHaveValidToString() {
             assertEquals("translator[h:gremlin-groovy]", 
GroovyTranslator.of("h").toString());
         }
    +
    +    @Test
    +    @LoadGraphWith(LoadGraphWith.GraphData.MODERN)
    --- End diff --
    
    It doesn't look like you need the "modern" graph data here - you can remove 
this annotation.


---

Reply via email to