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,7 +118,7 @@ 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> = [];
@@ -172,18 +172,14 @@ function collectFieldsImpl(
     runtimeType,
     operation,
     visitedFragmentNames,
+    maskSuggestions,
   } = context;
 
   for (const selection of selectionSet.selections) {
     switch (selection.kind) {
       case Kind.FIELD: {
         if (
-          !shouldIncludeNode(
-            selection,
-            variableValues,
-            fragmentVariableValues,
-            context.shouldProvideSuggestions,
-          )
+          !shouldIncludeNode(selection, variableValues, fragmentVariableValues)
         ) {
           continue;
         }
@@ -200,7 +196,6 @@ function collectFieldsImpl(
             selection,
             variableValues,
             fragmentVariableValues,
-            context.shouldProvideSuggestions,
           ) ||
           !doesFragmentConditionMatch(schema, selection, runtimeType)
         ) {
@@ -213,7 +208,6 @@ function collectFieldsImpl(
           fragmentVariableValues,
           selection,
           deferUsage,
-          context.shouldProvideSuggestions,
         );
 
         if (!newDeferUsage) {
@@ -248,7 +242,6 @@ function collectFieldsImpl(
           fragmentVariableValues,
           selection,
           deferUsage,
-          context.shouldProvideSuggestions,
         );
 
         if (
@@ -258,7 +251,6 @@ function collectFieldsImpl(
               selection,
               variableValues,
               fragmentVariableValues,
-              context.shouldProvideSuggestions,
             ))
         ) {
           continue;
@@ -279,8 +271,8 @@ function collectFieldsImpl(
             selection,
             fragmentVariableSignatures,
             variableValues,
-            context.shouldProvideSuggestions,
             fragmentVariableValues,
+            maskSuggestions,
           );
         }
 
@@ -316,19 +308,16 @@ function collectFieldsImpl(
  * deferred based on the experimental flag, defer directive present and
  * not disabled by the "if" argument.
  */
-// eslint-disable-next-line @typescript-eslint/max-params
 function getDeferUsage(
   operation: OperationDefinitionNode,
   variableValues: VariableValues,
   fragmentVariableValues: VariableValues | undefined,
   node: FragmentSpreadNode | InlineFragmentNode,
   parentDeferUsage: DeferUsage | undefined,
-  shouldProvideSuggestions: boolean,
 ): DeferUsage | undefined {
   const defer = getDirectiveValues(
     GraphQLDeferDirective,
     node,
-    shouldProvideSuggestions,
     variableValues,
     fragmentVariableValues,
   );
@@ -360,12 +349,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 +363,6 @@ function shouldIncludeNode(
   const include = getDirectiveValues(
     GraphQLIncludeDirective,
     node,
-    shouldProvideSuggestions,
     variableValues,
     fragmentVariableValues,
   );

src/execution/execute.ts

@@ -98,21 +98,16 @@ const collectSubfields = memoize3(
     returnType: GraphQLObjectType,
     fieldDetailsList: FieldDetailsList,
   ) => {
-    const {
-      schema,
-      fragments,
-      operation,
-      variableValues,
-      shouldProvideSuggestions,
-    } = validatedExecutionArgs;
+    const { schema, fragments, operation, variableValues, maskSuggestions } =
+      validatedExecutionArgs;
     return _collectSubfields(
       schema,
       fragments,
       variableValues,
       operation,
       returnType,
       fieldDetailsList,
-      shouldProvideSuggestions,
+      maskSuggestions,
     );
   },
 );
@@ -161,7 +156,7 @@ export interface ValidatedExecutionArgs {
     validatedExecutionArgs: ValidatedExecutionArgs,
   ) => PromiseOrValue<ExecutionResult>;
   enableEarlyExecution: boolean;
-  shouldProvideSuggestions: boolean;
+  maskSuggestions: boolean;
 }
 
 export interface ExecutionContext {
@@ -191,7 +186,7 @@ export interface ExecutionArgs {
     ) => PromiseOrValue<ExecutionResult>
   >;
   enableEarlyExecution?: Maybe<boolean>;
-  shouldProvideSuggestions?: Maybe<boolean>;
+  maskSuggestions?: Maybe<boolean>;
 }
 
 export interface StreamUsage {
@@ -322,7 +317,7 @@ export function experimentalExecuteQueryOrMutationOrSubscriptionEvent(
       rootValue,
       operation,
       variableValues,
-      shouldProvideSuggestions,
+      maskSuggestions,
     } = validatedExecutionArgs;
     const rootType = schema.getRootType(operation.operation);
     if (rootType == null) {
@@ -338,7 +333,7 @@ export function experimentalExecuteQueryOrMutationOrSubscriptionEvent(
       variableValues,
       rootType,
       operation,
-      shouldProvideSuggestions,
+      maskSuggestions,
     );
 
     const { groupedFieldSet, newDeferUsages } = collectedFields;
@@ -516,7 +511,6 @@ export function validateExecutionArgs(
     subscribeFieldResolver,
     perEventExecutor,
     enableEarlyExecution,
-    shouldProvideSuggestions,
   } = args;
 
   // If the schema used for execution is invalid, throw an error.
@@ -570,14 +564,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 +593,7 @@ export function validateExecutionArgs(
     subscribeFieldResolver: subscribeFieldResolver ?? defaultFieldResolver,
     perEventExecutor: perEventExecutor ?? executeSubscriptionEvent,
     enableEarlyExecution: enableEarlyExecution === true,
-    shouldProvideSuggestions: shouldProvideSuggestions ?? true,
+    maskSuggestions,
   };
 }
 
@@ -782,7 +777,8 @@ 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 +805,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,14 +1110,12 @@ function getStreamUsage(
       ._streamUsage;
   }
 
-  const { operation, variableValues, shouldProvideSuggestions } =
-    validatedExecutionArgs;
+  const { operation, variableValues } = 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,
   );
@@ -2088,7 +2082,7 @@ function executeSubscription(
     contextValue,
     operation,
     variableValues,
-    shouldProvideSuggestions,
+    maskSuggestions,
   } = validatedExecutionArgs;
 
   const rootType = schema.getSubscriptionType();
@@ -2105,7 +2099,7 @@ function executeSubscription(
     variableValues,
     rootType,
     operation,
-    shouldProvideSuggestions,
+    maskSuggestions,
   );
 
   const firstRootField = groupedFieldSet.entries().next().value as [
@@ -2142,8 +2136,8 @@ function executeSubscription(
     const args = getArgumentValues(
       fieldDef,
       fieldNodes[0],
-      validatedExecutionArgs.shouldProvideSuggestions,
       variableValues,
+      maskSuggestions,
     );
 
     // Call the `subscribe()` resolver or the default resolver to produce an