refactor(IncrementalGraph): remove unnecessary method

Author: yaacovCR

src/execution/IncrementalGraph.ts

@@ -74,17 +74,16 @@ export class IncrementalGraph {
   }
 
   getNewPending(): ReadonlyArray<SubsequentResultRecord> {
-    const maybeEmptyNewPending = this._newPending;
     const newPending = [];
-    for (const node of maybeEmptyNewPending) {
+    for (const node of this._newPending) {
       if (isDeferredFragmentRecord(node)) {
         if (node.expectedReconcilableResults) {
           this._pending.add(node);
           newPending.push(node);
           continue;
         }
         for (const child of node.children) {
-          this._addNonEmptyNewPending(child, newPending);
+          this._newPending.add(child);
         }
       } else {
         this._pending.add(node);
@@ -188,22 +187,6 @@ export class IncrementalGraph {
     this._addDeferredFragmentRecord(parent);
   }
 
-  private _addNonEmptyNewPending(
-    deferredFragmentRecord: DeferredFragmentRecord,
-    newPending: Array<SubsequentResultRecord>,
-  ): void {
-    if (deferredFragmentRecord.expectedReconcilableResults) {
-      this._pending.add(deferredFragmentRecord);
-      newPending.push(deferredFragmentRecord);
-      return;
-    }
-    /* c8 ignore next 5 */
-    // TODO: add test case for this, if when skipping an empty deferred fragment, the empty fragment has nested children.
-    for (const child of deferredFragmentRecord.children) {
-      this._addNonEmptyNewPending(child, newPending);
-    }
-  }
-
   private _enqueueCompletedDeferredGroupedFieldSet(
     result: DeferredGroupedFieldSetResult,
   ): void {