Remove unnecessary Promise.resolve and Promise.reject

graphql · Oct 6, 2021 · fcdeaf9 · fcdeaf9
1 parent bfc354e
commit fcdeaf9
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 4 deletions.
diff --git a/src/execution/execute.ts b/src/execution/execute.ts
@@ -368,7 +368,7 @@ function executeOperation(
     if (isPromise(result)) {
       return result.then(undefined, (error) => {
         exeContext.errors.push(error);
-        return Promise.resolve(null);
+        return null;
       });
     }
     return result;

diff --git a/src/subscription/mapAsyncIterator.ts b/src/subscription/mapAsyncIterator.ts
@@ -43,9 +43,10 @@ export function mapAsyncIterator<T, U, R = undefined>(
         : { value: undefined as any, done: true };
     },
     async throw(error?: unknown) {
-      return typeof iterator.throw === 'function'
-        ? mapResult(await iterator.throw(error))
-        : Promise.reject(error);
+      if (typeof iterator.throw === 'function') {
+        return mapResult(await iterator.throw(error));
+      }
+      throw error;
     },
     [Symbol.asyncIterator]() {
       return this;