introduce addError instead of handleFieldError

addError calls locatedError and then adds the error as previously in
handleFieldError, throwing if the type is non-null

Author: yaacovCR

src/execution/execute.ts

@@ -708,8 +708,13 @@ function executeField(
     result = resolveFn(source, args, contextValue, info);
   } catch (rawError) {
     const errors = asyncPayloadRecord?.errors ?? exeContext.errors;
-    const error = locatedError(rawError, fieldNodes, pathToArray(path));
-    const handledError = handleFieldError(error, returnType, errors);
+    const handledError = addError(
+      rawError,
+      fieldNodes,
+      returnType,
+      path,
+      errors,
+    );
     filterSubsequentPayloads(exeContext, path, asyncPayloadRecord);
     return handledError;
   }
@@ -762,8 +767,13 @@ function completePromiseCatchingErrors(
       // Note: we don't rely on a `catch` method, but we do expect "thenable"
       // to take a second callback for the error case.
       const errors = asyncPayloadRecord?.errors ?? exeContext.errors;
-      const error = locatedError(rawError, fieldNodes, pathToArray(path));
-      const handledError = handleFieldError(error, returnType, errors);
+      const handledError = addError(
+        rawError,
+        fieldNodes,
+        returnType,
+        path,
+        errors,
+      );
       filterSubsequentPayloads(exeContext, path, asyncPayloadRecord);
       return handledError;
     });
@@ -791,8 +801,13 @@ function completeValueCatchingErrors(
     );
   } catch (rawError) {
     const errors = asyncPayloadRecord?.errors ?? exeContext.errors;
-    const error = locatedError(rawError, fieldNodes, pathToArray(path));
-    const handledError = handleFieldError(error, returnType, errors);
+    const handledError = addError(
+      rawError,
+      fieldNodes,
+      returnType,
+      path,
+      errors,
+    );
     filterSubsequentPayloads(exeContext, path, asyncPayloadRecord);
     return handledError;
   }
@@ -802,8 +817,13 @@ function completeValueCatchingErrors(
     // to take a second callback for the error case.
     completedItem = completedItem.then(undefined, (rawError) => {
       const errors = asyncPayloadRecord?.errors ?? exeContext.errors;
-      const error = locatedError(rawError, fieldNodes, pathToArray(path));
-      const handledError = handleFieldError(error, returnType, errors);
+      const handledError = addError(
+        rawError,
+        fieldNodes,
+        returnType,
+        path,
+        errors,
+      );
       filterSubsequentPayloads(exeContext, path, asyncPayloadRecord);
       return handledError;
     });
@@ -838,11 +858,15 @@ export function buildResolveInfo(
   };
 }
 
-function handleFieldError(
-  error: GraphQLError,
+function addError(
+  rawError: unknown,
+  fieldNodes: ReadonlyArray<FieldNode>,
   returnType: GraphQLOutputType,
+  path: Path,
   errors: Array<GraphQLError>,
 ): null {
+  const error = locatedError(rawError, fieldNodes, pathToArray(path));
+
   // If the field type is non-nullable, then it is resolved without any
   // protection from errors, however it still properly locates the error.
   if (isNonNullType(returnType)) {
@@ -1066,8 +1090,9 @@ async function completeAsyncIteratorValue(
       // eslint-disable-next-line no-await-in-loop
       iteration = await iterator.next();
     } catch (rawError) {
-      const error = locatedError(rawError, fieldNodes, pathToArray(itemPath));
-      completedResults.push(handleFieldError(error, itemType, errors));
+      completedResults.push(
+        addError(rawError, fieldNodes, itemType, itemPath, errors),
+      );
       break;
     }
 
@@ -2013,8 +2038,13 @@ async function executeStreamAsyncIteratorItem(
   try {
     iteration = await iterator.next();
   } catch (rawError) {
-    const error = locatedError(rawError, fieldNodes, pathToArray(itemPath));
-    const value = handleFieldError(error, itemType, asyncPayloadRecord.errors);
+    const value = addError(
+      rawError,
+      fieldNodes,
+      itemType,
+      itemPath,
+      asyncPayloadRecord.errors,
+    );
     // don't continue if iterator throws
     return { done: true, value };
   }