Add tests

Author: JoviDeCroock

src/execution/collectFields.ts

@@ -55,7 +55,7 @@ interface CollectFieldsContext {
   operation: OperationDefinitionNode;
   runtimeType: GraphQLObjectType;
   visitedFragmentNames: Set<string>;
-  shouldProvideSuggestions: boolean;
+  maskSuggestions: boolean
 }
 
 /**
@@ -74,7 +74,7 @@ export function collectFields(
   variableValues: VariableValues,
   runtimeType: GraphQLObjectType,
   operation: OperationDefinitionNode,
-  shouldProvideSuggestions: boolean,
+  maskSuggestions: boolean,
 ): {
   groupedFieldSet: GroupedFieldSet;
   newDeferUsages: ReadonlyArray<DeferUsage>;
@@ -88,7 +88,7 @@ export function collectFields(
     runtimeType,
     operation,
     visitedFragmentNames: new Set(),
-    shouldProvideSuggestions,
+    maskSuggestions,
   };
 
   collectFieldsImpl(
@@ -118,8 +118,8 @@ export function collectSubfields(
   operation: OperationDefinitionNode,
   returnType: GraphQLObjectType,
   fieldDetailsList: FieldDetailsList,
-  shouldProvideSuggestions: boolean,
-): {
+  maskSuggestions: boolean,
+  ): {
   groupedFieldSet: GroupedFieldSet;
   newDeferUsages: ReadonlyArray<DeferUsage>;
 } {
@@ -130,7 +130,7 @@ export function collectSubfields(
     runtimeType: returnType,
     operation,
     visitedFragmentNames: new Set(),
-    shouldProvideSuggestions,
+    maskSuggestions
   };
   const subGroupedFieldSet = new AccumulatorMap<string, FieldDetails>();
   const newDeferUsages: Array<DeferUsage> = [];
@@ -182,7 +182,6 @@ function collectFieldsImpl(
             selection,
             variableValues,
             fragmentVariableValues,
-            context.shouldProvideSuggestions,
           )
         ) {
           continue;
@@ -200,7 +199,6 @@ function collectFieldsImpl(
             selection,
             variableValues,
             fragmentVariableValues,
-            context.shouldProvideSuggestions,
           ) ||
           !doesFragmentConditionMatch(schema, selection, runtimeType)
         ) {
@@ -213,7 +211,6 @@ function collectFieldsImpl(
           fragmentVariableValues,
           selection,
           deferUsage,
-          context.shouldProvideSuggestions,
         );
 
         if (!newDeferUsage) {
@@ -248,7 +245,6 @@ function collectFieldsImpl(
           fragmentVariableValues,
           selection,
           deferUsage,
-          context.shouldProvideSuggestions,
         );
 
         if (
@@ -258,7 +254,6 @@ function collectFieldsImpl(
               selection,
               variableValues,
               fragmentVariableValues,
-              context.shouldProvideSuggestions,
             ))
         ) {
           continue;
@@ -279,7 +274,7 @@ function collectFieldsImpl(
             selection,
             fragmentVariableSignatures,
             variableValues,
-            context.shouldProvideSuggestions,
+            false,
             fragmentVariableValues,
           );
         }
@@ -323,12 +318,10 @@ function getDeferUsage(
   fragmentVariableValues: VariableValues | undefined,
   node: FragmentSpreadNode | InlineFragmentNode,
   parentDeferUsage: DeferUsage | undefined,
-  shouldProvideSuggestions: boolean,
 ): DeferUsage | undefined {
   const defer = getDirectiveValues(
     GraphQLDeferDirective,
     node,
-    shouldProvideSuggestions,
     variableValues,
     fragmentVariableValues,
   );
@@ -360,12 +353,10 @@ function shouldIncludeNode(
   node: FragmentSpreadNode | FieldNode | InlineFragmentNode,
   variableValues: VariableValues,
   fragmentVariableValues: VariableValues | undefined,
-  shouldProvideSuggestions: boolean,
 ): boolean {
   const skip = getDirectiveValues(
     GraphQLSkipDirective,
     node,
-    shouldProvideSuggestions,
     variableValues,
     fragmentVariableValues,
   );
@@ -376,7 +367,6 @@ function shouldIncludeNode(
   const include = getDirectiveValues(
     GraphQLIncludeDirective,
     node,
-    shouldProvideSuggestions,
     variableValues,
     fragmentVariableValues,
   );

src/execution/execute.ts

@@ -103,7 +103,7 @@ const collectSubfields = memoize3(
       fragments,
       operation,
       variableValues,
-      shouldProvideSuggestions,
+      maskSuggestions,
     } = validatedExecutionArgs;
     return _collectSubfields(
       schema,
@@ -112,7 +112,7 @@ const collectSubfields = memoize3(
       operation,
       returnType,
       fieldDetailsList,
-      shouldProvideSuggestions,
+      maskSuggestions,
     );
   },
 );
@@ -161,7 +161,7 @@ export interface ValidatedExecutionArgs {
     validatedExecutionArgs: ValidatedExecutionArgs,
   ) => PromiseOrValue<ExecutionResult>;
   enableEarlyExecution: boolean;
-  shouldProvideSuggestions: boolean;
+  maskSuggestions: boolean;
 }
 
 export interface ExecutionContext {
@@ -191,7 +191,7 @@ export interface ExecutionArgs {
     ) => PromiseOrValue<ExecutionResult>
   >;
   enableEarlyExecution?: Maybe<boolean>;
-  shouldProvideSuggestions?: Maybe<boolean>;
+  maskSuggestions?: Maybe<boolean>;
 }
 
 export interface StreamUsage {
@@ -322,7 +322,7 @@ export function experimentalExecuteQueryOrMutationOrSubscriptionEvent(
       rootValue,
       operation,
       variableValues,
-      shouldProvideSuggestions,
+      maskSuggestions,
     } = validatedExecutionArgs;
     const rootType = schema.getRootType(operation.operation);
     if (rootType == null) {
@@ -338,7 +338,7 @@ export function experimentalExecuteQueryOrMutationOrSubscriptionEvent(
       variableValues,
       rootType,
       operation,
-      shouldProvideSuggestions,
+      maskSuggestions,
     );
 
     const { groupedFieldSet, newDeferUsages } = collectedFields;
@@ -516,7 +516,6 @@ export function validateExecutionArgs(
     subscribeFieldResolver,
     perEventExecutor,
     enableEarlyExecution,
-    shouldProvideSuggestions,
   } = args;
 
   // If the schema used for execution is invalid, throw an error.
@@ -570,14 +569,15 @@ export function validateExecutionArgs(
   // FIXME: https://github.com/graphql/graphql-js/issues/2203
   /* c8 ignore next */
   const variableDefinitions = operation.variableDefinitions ?? [];
+  const maskSuggestions = args.maskSuggestions ?? false;
 
   const variableValuesOrErrors = getVariableValues(
     schema,
     variableDefinitions,
     rawVariableValues ?? {},
     {
       maxErrors: 50,
-      shouldProvideSuggestions: shouldProvideSuggestions ?? true,
+      maskSuggestions
     },
   );
 
@@ -598,7 +598,7 @@ export function validateExecutionArgs(
     subscribeFieldResolver: subscribeFieldResolver ?? defaultFieldResolver,
     perEventExecutor: perEventExecutor ?? executeSubscriptionEvent,
     enableEarlyExecution: enableEarlyExecution === true,
-    shouldProvideSuggestions: shouldProvideSuggestions ?? true,
+    maskSuggestions,
   };
 }
 
@@ -782,7 +782,7 @@ function executeField(
   deferMap: ReadonlyMap<DeferUsage, DeferredFragmentRecord> | undefined,
 ): PromiseOrValue<GraphQLWrappedResult<unknown>> | undefined {
   const validatedExecutionArgs = exeContext.validatedExecutionArgs;
-  const { schema, contextValue, variableValues, shouldProvideSuggestions } = validatedExecutionArgs;
+  const { schema, contextValue, variableValues, maskSuggestions } = validatedExecutionArgs;
   const fieldName = fieldDetailsList[0].node.name.value;
   const fieldDef = schema.getField(parentType, fieldName);
   if (!fieldDef) {
@@ -809,8 +809,8 @@ function executeField(
       fieldDetailsList[0].node,
       fieldDef.args,
       variableValues,
-      shouldProvideSuggestions,
       fieldDetailsList[0].fragmentVariableValues,
+      maskSuggestions,
     );
 
     // The resolve function's optional third argument is a context value that
@@ -1114,16 +1114,16 @@ function getStreamUsage(
       ._streamUsage;
   }
 
-  const { operation, variableValues, shouldProvideSuggestions } =
+  const { operation, variableValues, maskSuggestions } =
     validatedExecutionArgs;
   // validation only allows equivalent streams on multiple fields, so it is
   // safe to only check the first fieldNode for the stream directive
   const stream = getDirectiveValues(
     GraphQLStreamDirective,
     fieldDetailsList[0].node,
-    shouldProvideSuggestions,
     variableValues,
     fieldDetailsList[0].fragmentVariableValues,
+    maskSuggestions,
   );
 
   if (!stream) {
@@ -2088,7 +2088,7 @@ function executeSubscription(
     contextValue,
     operation,
     variableValues,
-    shouldProvideSuggestions,
+    maskSuggestions,
   } = validatedExecutionArgs;
 
   const rootType = schema.getSubscriptionType();
@@ -2105,7 +2105,7 @@ function executeSubscription(
     variableValues,
     rootType,
     operation,
-    shouldProvideSuggestions,
+    maskSuggestions,
   );
 
   const firstRootField = groupedFieldSet.entries().next().value as [
@@ -2142,8 +2142,8 @@ function executeSubscription(
     const args = getArgumentValues(
       fieldDef,
       fieldNodes[0],
-      validatedExecutionArgs.shouldProvideSuggestions,
       variableValues,
+      validatedExecutionArgs.maskSuggestions,
     );
 
     // Call the `subscribe()` resolver or the default resolver to produce an