print_ast: remove indentation inside of block strings

Replicates graphql/graphql-js@45e33ce

Author: Cito

src/graphql/language/block_string.py

@@ -69,9 +69,7 @@ def get_block_string_indentation(value: str) -> int:
     return common_indent or 0
 
 
-def print_block_string(
-    value: str, indentation: str = "", prefer_multiple_lines: bool = False
-) -> str:
+def print_block_string(value: str, prefer_multiple_lines: bool = False) -> str:
     """Print a block string in the indented block form.
 
     Prints a block string in the indented block form by adding a leading and
@@ -92,11 +90,11 @@ def print_block_string(
     )
 
     # Format a multi-line block quote to account for leading space.
-    if print_as_multiple_lines and not (is_single_line and has_leading_space):
-        result = "\n" + indentation
-    else:
-        result = ""
-    result += value.replace("\n", "\n" + indentation) if indentation else value
+    result = (
+        "\n"
+        if print_as_multiple_lines and not (is_single_line and has_leading_space)
+        else ""
+    ) + value
     if print_as_multiple_lines:
         result += "\n"
 

src/graphql/language/printer.py

@@ -154,9 +154,9 @@ def leave_float_value(node: PrintedNode, *_args: Any) -> str:
         return node.value
 
     @staticmethod
-    def leave_string_value(node: PrintedNode, key: str, *_args: Any) -> str:
+    def leave_string_value(node: PrintedNode, *_args: Any) -> str:
         if node.block:
-            return print_block_string(node.value, "" if key == "description" else "  ")
+            return print_block_string(node.value)
         return dumps(node.value)
 
     @staticmethod

src/graphql/utilities/print_schema.py

@@ -295,7 +295,7 @@ def print_description(
         return ""
 
     prefer_multiple_lines = len(description) > 70
-    block_string = print_block_string(description, "", prefer_multiple_lines)
+    block_string = print_block_string(description, prefer_multiple_lines)
 
     prefix = "\n" + indentation if indentation and not first_in_block else indentation
 

tests/language/test_block_string.py

@@ -116,28 +116,27 @@ def do_not_escape_characters():
     def by_default_print_block_strings_as_single_line():
         s = "one liner"
         assert print_block_string(s) == '"""one liner"""'
-        assert print_block_string(s, "", True) == '"""\none liner\n"""'
+        assert print_block_string(s, True) == '"""\none liner\n"""'
 
     def correctly_prints_single_line_with_leading_space():
         s = "    space-led string"
         assert print_block_string(s) == '"""    space-led string"""'
-        assert print_block_string(s, "", True) == '"""    space-led string\n"""'
+        assert print_block_string(s, True) == '"""    space-led string\n"""'
 
     def correctly_prints_single_line_with_leading_space_and_quotation():
         s = '    space-led value "quoted string"'
 
         assert print_block_string(s) == '"""    space-led value "quoted string"\n"""'
 
         assert (
-            print_block_string(s, "", True)
-            == '"""    space-led value "quoted string"\n"""'
+            print_block_string(s, True) == '"""    space-led value "quoted string"\n"""'
         )
 
     def correctly_prints_single_line_with_trailing_backslash():
         s = "backslash \\"
 
         assert print_block_string(s) == '"""\nbackslash \\\n"""'
-        assert print_block_string(s, "", True) == '"""\nbackslash \\\n"""'
+        assert print_block_string(s, True) == '"""\nbackslash \\\n"""'
 
     def correctly_prints_string_with_a_first_line_indentation():
         s = join_lines("    first  ", "  line     ", "indentation", "     string")

tests/language/test_block_string_fuzz.py

@@ -41,13 +41,11 @@ def correctly_print_random_strings():
                 """
             )
 
-            printed_multiline_string = lex_value(
-                print_block_string(test_value, " ", True)
-            )
+            printed_multiline_string = lex_value(print_block_string(test_value, True))
 
             assert test_value == printed_multiline_string, dedent(
                 f"""
-                Expected lex_value(print_block_string({test_value!r}, ' ', True)
+                Expected lex_value(print_block_string({test_value!r}, True)
                   to equal {test_value!r}
                   but got {printed_multiline_string!r}
                 """

tests/language/test_printer.py

@@ -189,7 +189,7 @@ def prints_kitchen_sink(kitchen_sink_query):  # noqa: F811
                 size: $size
                 bar: $b
                 obj: {key: "value", block: """
-                  block string uses \"""
+                block string uses \"""
                 """}
               )
             }