Merge pull request #2 from igrlk/fix-abort-execution

Fix aborted in tests
graphql · Oct 2, 2023 · 77e6176 · 77e6176
2 parents 1f851b3 + cb619c0
commit 77e6176
Showing 1 changed file with 4 additions and 17 deletions.
diff --git a/src/execution/execute.ts b/src/execution/execute.ts
@@ -262,28 +262,15 @@ function executeImpl(
     const data = executeOperation(exeContext, initialResultRecord);
     if (isPromise(data)) {
       return data.then(
-        (resolved) => {
-          exeContext.executionController.abort();
-          return incrementalPublisher.buildDataResponse(
-            initialResultRecord,
-            resolved,
-          );
-        },
-        (error) => {
-          exeContext.executionController.abort();
-          return incrementalPublisher.buildErrorResponse(
-            initialResultRecord,
-            error,
-          );
-        },
+        (resolved) =>
+          incrementalPublisher.buildDataResponse(initialResultRecord, resolved),
+        (error) =>
+          incrementalPublisher.buildErrorResponse(initialResultRecord, error),
       );
     }
 
-    exeContext.executionController.abort();
     return incrementalPublisher.buildDataResponse(initialResultRecord, data);
   } catch (error) {
-    exeContext.executionController.abort();
-
     return incrementalPublisher.buildErrorResponse(initialResultRecord, error);
   }
 }