[GitHub] tinkerpop pull request #934: Apply StringEscapeUtils in GroovyTranslator#con...

[justinchuch]

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

    https://github.com/apache/tinkerpop/pull/934#discussion_r219676264
  
    --- 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 --
    
    Thanks for pointing that out. I have combined the tests.


---