Append newline to printed documents

src/GraphQLParser.Tests/Visitors/SDLPrinterFromManualASTTests.cs

@@ -148,6 +148,7 @@ directive @null_locations on
             directive @empty_locations on
 
             scalar AAA
+
             """);
     }
 
@@ -186,6 +187,7 @@ enum EnumWithValuesDefinitionOfNullItems {
 
             enum EnumWithValuesDefinitionOfEmptyItems {
             }
+
             """);
     }
 
@@ -224,6 +226,7 @@ input InputWithFieldsDefinitionOfNullItems {
 
             input InputWithFieldsDefinitionOfEmptyItems {
             }
+
             """);
     }
 
@@ -262,6 +265,7 @@ type TypeWithFieldsDefinitionOfNullItems {
 
             type TypeWithFieldsDefinitionOfEmptyItems {
             }
+
             """);
     }
 }

src/GraphQLParser.Tests/Visitors/SDLPrinterFromParsedTextTests.cs

@@ -869,7 +869,7 @@ public async Task SDLPrinter_Should_Print_Document(
 
         await printer.PrintAsync(document, writer).ConfigureAwait(false);
         var actual = writer.ToString();
-        actual.ShouldBe(expected, $"Test {number} failed");
+        actual.ShouldBe(expected + Environment.NewLine, $"Test {number} failed");
 
         actual.Parse(); // should be parsed back
     }
@@ -928,7 +928,7 @@ public async Task SDLPrinter_Should_Print_BlockStrings(int number, string input,
         var renderedOriginal = writer.ToString();
 
         var lines = renderedOriginal.Split(Environment.NewLine);
-        var renderedDescription = string.Join(Environment.NewLine, lines.SkipLast(1));
+        var renderedDescription = string.Join(Environment.NewLine, lines.SkipLast(2));
         renderedDescription = renderedDescription.Replace("\r\n", "\n");
         renderedDescription.ShouldBe(expected);
 
@@ -947,9 +947,12 @@ public async Task SDLPrinter_Should_Print_BlockStrings(int number, string input,
     public async Task SDLPrinter_Should_Print_EscapedStrings(string stringValue)
     {
         string query = $"{{a(p:{stringValue})}}";
-        string expected = @$"{{
-  a(p: {stringValue})
-}}";
+        string expected = $$"""
+            {
+              a(p: {{stringValue}})
+            }
+
+            """;
         var writer = new StringWriter();
 
         var document = query.Parse();

src/GraphQLParser.Tests/Visitors/SDLPrinterSkipDirectivesTests.cs

@@ -14,7 +14,8 @@ public class SDLPrinterSkipDirectivesTests
 @"type Foo {
   f: Int @aliased
   k: Boolean
-}")]
+}
+")]
     [InlineData(2,
 @"type Foo {
   f: Int @bad @aliased
@@ -24,7 +25,8 @@ public class SDLPrinterSkipDirectivesTests
 @"type Foo {
   f: Int @aliased
   k: Boolean
-}")]
+}
+")]
     [InlineData(3,
 @"type Foo {
   f: Int @aliased @bad
@@ -34,7 +36,8 @@ public class SDLPrinterSkipDirectivesTests
 @"type Foo {
   f: Int @aliased
   k: Boolean
-}")]
+}
+")]
     [InlineData(4,
 @"type Foo {
   f: Int @bad
@@ -44,11 +47,12 @@ public class SDLPrinterSkipDirectivesTests
 @"type Foo {
   f: Int
   k: Boolean
-}")]
+}
+")]
     public async Task Printer_Should_Print_Pretty_If_Directives_Skipped(
-int number,
-string text,
-string expected)
+        int number,
+        string text,
+        string expected)
     {
         var printer = new MyPrinter();
         var writer = new StringWriter();

src/GraphQLParser.Tests/Visitors/SDLPrinterVerticalIndentationTests.cs

@@ -178,17 +178,17 @@ extend type A2 {
   a: Int
 }")]
     public async Task Printer_Should_Print_Pretty_If_Definitions_Skipped(
-int number,
-string text,
-string expected)
+        int number,
+        string text,
+        string expected)
     {
         var printer = new PrintOnlyStartsWithA();
         var writer = new StringWriter();
         var document = text.Parse();
 
         await printer.PrintAsync(document, writer).ConfigureAwait(false);
         var actual = writer.ToString();
-        actual.ShouldBe(expected, $"Test {number} failed");
+        actual.ShouldBe(expected + Environment.NewLine, $"Test {number} failed");
 
         actual.Parse(); // should be parsed back
     }

src/GraphQLParser.Tests/Visitors/SDLSorterTests.SortsSchemaDefinition.approved.DefinitionSortTests.graphql

@@ -27,4 +27,4 @@ input Type2 {
   f3: ID!
 }
 
-scalar Type3
+scalar Type3

src/GraphQLParser.Tests/Visitors/SDLSorterTests.SortsSchemaDefinition.approved.ExecutableSortTests.graphql

@@ -65,4 +65,4 @@ fragment frag3 on Type3 {
   dummy1
   dummy2
   dummy3
-}
+}

src/GraphQLParser.Tests/Visitors/SDLSorterTests.SortsSchemaDefinition.approved.KitchenSink.graphql

@@ -139,4 +139,4 @@ extend type Foo {
   seven(argument: [String]): Type
 }
 
-extend type Foo @onType
+extend type Foo @onType

src/GraphQLParser/Visitors/SDLPrinter.cs

@@ -1097,13 +1097,15 @@ public SDLPrinter(SDLPrinterOptions options)
     }
 
     /// <inheritdoc cref="SDLPrinter{TContext}"/>
-    public virtual ValueTask PrintAsync(ASTNode node, TextWriter writer, CancellationToken cancellationToken = default)
+    public virtual async ValueTask PrintAsync(ASTNode node, TextWriter writer, CancellationToken cancellationToken = default)
     {
         var context = new DefaultPrintContext(writer)
         {
             CancellationToken = cancellationToken,
         };
-        return VisitAsync(node, context);
+        await VisitAsync(node, context).ConfigureAwait(false);
+        if (!context.NewLinePrinted && node is GraphQLDocument)
+            await writer.WriteLineAsync().ConfigureAwait(false);
     }
 }