fix test coverage

TODO: add back test
graphql · Dec 17, 2023 · d10bf74 · d10bf74
1 parent 026044f
commit d10bf74
Showing 1 changed file with 8 additions and 8 deletions.
diff --git a/src/execution/IncrementalPublisher.ts b/src/execution/IncrementalPublisher.ts
@@ -587,14 +587,14 @@ export class IncrementalPublisher {
     let maxLength: number | undefined;
     let idWithLongestPath: string | undefined;
     for (const deferredFragmentRecord of deferredFragmentRecords) {
-      const id = deferredFragmentRecord.id;
-      if (id === undefined) {
-        continue;
-      }
-      const length = deferredFragmentRecord.path.length;
-      if (maxLength === undefined || length > maxLength) {
-        maxLength = length;
-        idWithLongestPath = id;
+      if (deferredFragmentRecord.id !== undefined) {
+        // pending must have been sent
+        const id = deferredFragmentRecord.id;
+        const length = deferredFragmentRecord.path.length;
+        if (maxLength === undefined || length > maxLength) {
+          maxLength = length;
+          idWithLongestPath = id;
+        }
       }
     }
     const subPath = deferredGroupedFieldSetRecord.path.slice(maxLength);