Fixed issue in execution node for subscriptions (graphql-dotnet#815)

BenjaBobs · joemcbride · commit dcb7b664a57b · 2018-09-10T07:28:35.000-07:00
Not having the parent becomes an issue further down the chain, causing a NullRefException in InstrumentFieldsMiddleware.Resolve(), resulting in a null-object been sent to the subscribing client. This fixes that.
diff --git a/src/GraphQL/Execution/SubscriptionExecutionStrategy.cs b/src/GraphQL/Execution/SubscriptionExecutionStrategy.cs
@@ -101,7 +101,7 @@ protected virtual async Task<IObservable<ExecutionResult>> ResolveEventStreamAsy
                 }
 
                 return subscription
-                    .Select(value => new ObjectExecutionNode(null, node.GraphType, node.Field, node.FieldDefinition, node.Path)
+                    .Select(value => new ObjectExecutionNode(node.Parent, node.GraphType, node.Field, node.FieldDefinition, node.Path)
                     {
                         Source = value
                     })

Original file line number	Diff line number	Diff line change
`@@ -101,7 +101,7 @@ protected virtual async Task<IObservable<ExecutionResult>> ResolveEventStreamAsy`
`101`	`101`	`}`
`102`	`102`
`103`	`103`	`return subscription`
`104`		`- .Select(value => new ObjectExecutionNode(null, node.GraphType, node.Field, node.FieldDefinition, node.Path)`
	`104`	`+ .Select(value => new ObjectExecutionNode(node.Parent, node.GraphType, node.Field, node.FieldDefinition, node.Path)`
`105`	`105`	`{`
`106`	`106`	`Source = value`
`107`	`107`	`})`