Skip to content

Commit b8a9c83

Browse files
committed
Fix SDLPrinter to include operation type for queries with variables
1 parent bd8fb7a commit b8a9c83

File tree

2 files changed

+2
-1
lines changed

2 files changed

+2
-1
lines changed

src/GraphQLParser.Tests/Visitors/SDLPrinterFromParsedTextTests.cs

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1066,6 +1066,7 @@ public void UTF8_MemoryStream_Runs_Synchronously()
10661066
[Theory]
10671067
[InlineData("{ field1 }", "{\n field1\n}\n")]
10681068
[InlineData("query { field1 }", "{\n field1\n}\n")]
1069+
[InlineData("query ($arg: String) { field1 }", "query ($arg: String){\n field1\n}\n")]
10691070
[InlineData("query q1 { field1 }", "query q1 {\n field1\n}\n")]
10701071
[InlineData("mutation { field1 }", "mutation {\n field1\n}\n")]
10711072
[InlineData("mutation m1 { field1 }", "mutation m1 {\n field1\n}\n")]

src/GraphQLParser/Visitors/SDLPrinter.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -262,7 +262,7 @@ protected override async ValueTask VisitFieldAsync(GraphQLField field, TContext
262262
protected override async ValueTask VisitOperationDefinitionAsync(GraphQLOperationDefinition operationDefinition, TContext context)
263263
{
264264
await VisitAsync(operationDefinition.Comments, context).ConfigureAwait(false);
265-
if (operationDefinition.Name is not null || operationDefinition.Operation != OperationType.Query)
265+
if (operationDefinition.Name is not null || operationDefinition.Operation != OperationType.Query || operationDefinition.Variables is not null)
266266
{
267267
await VisitAsync(LiteralNode.Wrap(GetOperationType(operationDefinition.Operation)), context).ConfigureAwait(false);
268268
await context.WriteAsync(" ").ConfigureAwait(false);

0 commit comments

Comments
 (0)