Fix AstPrinter trimming blank string values (#2672)

gnawf · web-flow · commit 2d522edea485 · 2022-01-12T10:34:55.000+11:00
diff --git a/src/main/java/graphql/language/AstPrinter.java b/src/main/java/graphql/language/AstPrinter.java
@@ -482,7 +482,7 @@ private String value(Value value) {
         } else if (value instanceof FloatValue) {
             return valueOf(((FloatValue) value).getValue());
         } else if (value instanceof StringValue) {
-            return wrap("\"", escapeJsonString(((StringValue) value).getValue()), "\"");
+            return "\"" + escapeJsonString(((StringValue) value).getValue()) + "\"";
         } else if (value instanceof EnumValue) {
             return valueOf(((EnumValue) value).getName());
         } else if (value instanceof BooleanValue) {
diff --git a/src/test/groovy/graphql/language/AstPrinterTest.groovy b/src/test/groovy/graphql/language/AstPrinterTest.groovy
@@ -402,6 +402,26 @@ query NullEpisodeQuery {
 '''
     }
 
+    def "ast printing of blank string"() {
+        def query = '''
+query NullEpisodeQuery {
+  human(id: "     ") {
+    name
+  }
+}
+'''
+        def document = parse(query)
+        String output = printAst(document)
+
+        expect:
+        output == '''query NullEpisodeQuery {
+  human(id: "     ") {
+    name
+  }
+}
+'''
+    }
+
     //-------------------------------------------------
     def "ast printing of default variables with null"() {
         def query = '''
