chore(server): add GQL error messages to bad subscription error

Garrett Moseke · Garrett Moseke · commit 020b560fe7a3 · 2022-04-07T14:00:49.000-06:00
diff --git a/Sources/GraphQLWS/GraphQLWSError.swift b/Sources/GraphQLWS/GraphQLWSError.swift
@@ -1,4 +1,5 @@
 // Copyright (c) 2021 PassiveLogic, Inc.
+import GraphQL
 
 struct GraphQLWSError: Error {
     let message: String
@@ -72,9 +73,9 @@ struct GraphQLWSError: Error {
         )
     }
     
-    static func internalAPIStreamIssue() -> Self {
+    static func internalAPIStreamIssue(errors: [GraphQLError]) -> Self {
         return self.init(
-            "API Response did not result in a stream type",
+            "API Response did not result in a stream type, contained errors\n\(errors.map { $0.message}.joined(separator: "\n"))",
             code: .internalAPIStreamIssue
         )
     }
diff --git a/Sources/GraphQLWS/Server.swift b/Sources/GraphQLWS/Server.swift
@@ -155,7 +155,7 @@ public class Server {
             subscribeFuture.whenSuccess { result in
                 guard let streamOpt = result.stream else {
                     // API issue - subscribe resolver isn't stream
-                    self.error(.internalAPIStreamIssue())
+                    self.error(.internalAPIStreamIssue(errors: result.errors))
                     return
                 }
                 let stream = streamOpt as! ObservableSubscriptionEventStream

Original file line number	Diff line number	Diff line change
`@@ -1,4 +1,5 @@`
`1`	`1`	`// Copyright (c) 2021 PassiveLogic, Inc.`
	`2`	`+import GraphQL`
`2`	`3`
`3`	`4`	`struct GraphQLWSError: Error {`
`4`	`5`	`let message: String`
`@@ -72,9 +73,9 @@ struct GraphQLWSError: Error {`
`72`	`73`	`)`
`73`	`74`	`}`
`74`	`75`
`75`		`- static func internalAPIStreamIssue() -> Self {`
	`76`	`+ static func internalAPIStreamIssue(errors: [GraphQLError]) -> Self {`
`76`	`77`	`return self.init(`
`77`		`- "API Response did not result in a stream type",`
	`78`	`+ "API Response did not result in a stream type, contained errors\n\(errors.map { $0.message}.joined(separator: "\n"))",`
`78`	`79`	`code: .internalAPIStreamIssue`
`79`	`80`	`)`
`80`	`81`	`}`
Original file line number	Diff line number	Diff line change
`@@ -155,7 +155,7 @@ public class Server {`
`155`	`155`	`subscribeFuture.whenSuccess { result in`
`156`	`156`	`guard let streamOpt = result.stream else {`
`157`	`157`	`// API issue - subscribe resolver isn't stream`
`158`		`- self.error(.internalAPIStreamIssue())`
	`158`	`+ self.error(.internalAPIStreamIssue(errors: result.errors))`
`159`	`159`	`return`
`160`	`160`	`}`
`161`	`161`	`let stream = streamOpt as! ObservableSubscriptionEventStream`