Change test to handle missing id case

apollographql · Oct 16, 2024 · e9ec645 · e9ec645
1 parent 079d1bd
commit e9ec645
Showing 1 changed file with 4 additions and 4 deletions.
diff --git a/Tests/ApolloTests/WebSocket/WebSocketTests.swift b/Tests/ApolloTests/WebSocket/WebSocketTests.swift
@@ -110,8 +110,8 @@ class WebSocketTests: XCTestCase {
     subject.cancel()
   }
 
-  func testLocalErrorUnknownId() throws {
-    let expectation = self.expectation(description: "Unknown id for subscription")
+  func testLocalErrorMissingId() throws {
+    let expectation = self.expectation(description: "Missing id for subscription")
 
     let subject = client.subscribe(subscription: MockSubscription<ReviewAddedData>()) { result in
       defer { expectation.fulfill() }
@@ -133,10 +133,10 @@ class WebSocketTests: XCTestCase {
         }
       }
     }
-
+
+    // Message data below has missing 'id' and should notify all subscribers of the error
     let message : JSONEncodableDictionary = [
       "type": "data",
-      "id": "2",            // subscribing on id = 1, i.e. expecting error when receiving id = 2
       "payload": [
         "data": [
           "reviewAdded": [