From b122456977b5e461264e27b39c03b112a5b6fbfd Mon Sep 17 00:00:00 2001
From: Ivan Goncharov <ivan.goncharov.ua@gmail.com>
Date: Sun, 21 Feb 2021 16:34:56 +0200
Subject: [PATCH] validateSchema: inline 'getAllNodes' function

---
 src/type/validate.js | 34 +++++++++-------------------------
 1 file changed, 9 insertions(+), 25 deletions(-)

diff --git a/src/type/validate.js b/src/type/validate.js
index 664c57332a5..b44c4c0dcd0 100644
--- a/src/type/validate.js
+++ b/src/type/validate.js
@@ -143,13 +143,9 @@ function getOperationTypeNode(
   schema: GraphQLSchema,
   operation: OperationTypeNode,
 ): ?ASTNode {
-  const operationNodes = getAllSubNodes(schema, (node) => node.operationTypes);
-  for (const node of operationNodes) {
-    if (node.operation === operation) {
-      return node.type;
-    }
-  }
-  return undefined;
+  return getAllNodes(schema)
+    .flatMap((schemaNode) => schemaNode.operationTypes ?? [])
+    .find((operationNode) => operationNode.operation === operation).type;
 }
 
 function validateDirectives(context: SchemaValidationContext): void {
@@ -626,34 +622,22 @@ function getAllNodes<T: ASTNode, K: ASTNode>(
     : extensionASTNodes ?? [];
 }
 
-function getAllSubNodes<T: ASTNode, K: ASTNode, L: ASTNode>(
-  object: SDLDefinedObject<T, K>,
-  getter: (T | K) => ?(L | $ReadOnlyArray<L>),
-): $ReadOnlyArray<L> {
-  let subNodes = [];
-  for (const node of getAllNodes(object)) {
-    // istanbul ignore next (See: 'https://github.com/graphql/graphql-js/issues/2203')
-    subNodes = subNodes.concat(getter(node) ?? []);
-  }
-  return subNodes;
-}
-
 function getAllImplementsInterfaceNodes(
   type: GraphQLObjectType | GraphQLInterfaceType,
   iface: GraphQLInterfaceType,
 ): $ReadOnlyArray<NamedTypeNode> {
-  return getAllSubNodes(type, (typeNode) => typeNode.interfaces).filter(
-    (ifaceNode) => ifaceNode.name.value === iface.name,
-  );
+  return getAllNodes(type)
+    .flatMap((typeNode) => typeNode.interfaces ?? [])
+    .filter((ifaceNode) => ifaceNode.name.value === iface.name);
 }
 
 function getUnionMemberTypeNodes(
   union: GraphQLUnionType,
   typeName: string,
 ): ?$ReadOnlyArray<NamedTypeNode> {
-  return getAllSubNodes(union, (unionNode) => unionNode.types).filter(
-    (typeNode) => typeNode.name.value === typeName,
-  );
+  return getAllNodes(union)
+    .flatMap((unionNode) => unionNode.types ?? [])
+    .filter((typeNode) => typeNode.name.value === typeName);
 }
 
 function getDeprecatedDirectiveNode(