[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_r219678287
  
    --- 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)
    +    public void shouldHandleTraversalAddV() {
    --- End diff --
    
    I have renamed the test method and separate into two:
    1. `shouldEscapeStrings`
    2. `shouldHandleVertexAndEdge`


---