validation: Fix names of internal SDL rules (#2448)

Not a breaking change since these rules are internal.

Author: IvanGoncharov

src/validation/__tests__/KnownArgumentNamesRule-test.js

@@ -6,7 +6,7 @@ import { buildSchema } from '../../utilities/buildASTSchema';
 
 import {
   KnownArgumentNamesRule,
-  KnownArgumentNamesOnDirectives,
+  KnownArgumentNamesOnDirectivesRule,
 } from '../rules/KnownArgumentNamesRule';
 
 import { expectValidationErrors, expectSDLValidationErrors } from './harness';
@@ -22,7 +22,7 @@ function expectValid(queryStr) {
 function expectSDLErrors(sdlStr, schema) {
   return expectSDLValidationErrors(
     schema,
-    KnownArgumentNamesOnDirectives,
+    KnownArgumentNamesOnDirectivesRule,
     sdlStr,
   );
 }

src/validation/__tests__/ProvidedRequiredArgumentsRule-test.js

@@ -6,7 +6,7 @@ import { buildSchema } from '../../utilities/buildASTSchema';
 
 import {
   ProvidedRequiredArgumentsRule,
-  ProvidedRequiredArgumentsOnDirectives,
+  ProvidedRequiredArgumentsOnDirectivesRule,
 } from '../rules/ProvidedRequiredArgumentsRule';
 
 import { expectValidationErrors, expectSDLValidationErrors } from './harness';
@@ -22,7 +22,7 @@ function expectValid(queryStr) {
 function expectSDLErrors(sdlStr, schema) {
   return expectSDLValidationErrors(
     schema,
-    ProvidedRequiredArgumentsOnDirectives,
+    ProvidedRequiredArgumentsOnDirectivesRule,
     sdlStr,
   );
 }

src/validation/rules/KnownArgumentNamesRule.d.ts

@@ -12,6 +12,6 @@ export function KnownArgumentNamesRule(context: ValidationContext): ASTVisitor;
 /**
  * @internal
  */
-export function KnownArgumentNamesOnDirectives(
+export function KnownArgumentNamesOnDirectivesRule(
   context: ValidationContext | SDLValidationContext,
 ): ASTVisitor;

src/validation/rules/KnownArgumentNamesRule.js

@@ -23,7 +23,7 @@ import {
  */
 export function KnownArgumentNamesRule(context: ValidationContext): ASTVisitor {
   return {
-    ...KnownArgumentNamesOnDirectives(context),
+    ...KnownArgumentNamesOnDirectivesRule(context),
     Argument(argNode) {
       const argDef = context.getArgument();
       const fieldDef = context.getFieldDef();
@@ -48,7 +48,7 @@ export function KnownArgumentNamesRule(context: ValidationContext): ASTVisitor {
 /**
  * @internal
  */
-export function KnownArgumentNamesOnDirectives(
+export function KnownArgumentNamesOnDirectivesRule(
   context: ValidationContext | SDLValidationContext,
 ): ASTVisitor {
   const directiveArgs = Object.create(null);

src/validation/rules/ProvidedRequiredArgumentsRule.d.ts

@@ -14,6 +14,6 @@ export function ProvidedRequiredArgumentsRule(
 /**
  * @internal
  */
-export function ProvidedRequiredArgumentsOnDirectives(
+export function ProvidedRequiredArgumentsOnDirectivesRule(
   context: ValidationContext | SDLValidationContext,
 ): ASTVisitor;

src/validation/rules/ProvidedRequiredArgumentsRule.js

@@ -27,7 +27,7 @@ export function ProvidedRequiredArgumentsRule(
   context: ValidationContext,
 ): ASTVisitor {
   return {
-    ...ProvidedRequiredArgumentsOnDirectives(context),
+    ...ProvidedRequiredArgumentsOnDirectivesRule(context),
     Field: {
       // Validate on leave to allow for deeper errors to appear first.
       leave(fieldNode) {
@@ -59,7 +59,7 @@ export function ProvidedRequiredArgumentsRule(
 /**
  * @internal
  */
-export function ProvidedRequiredArgumentsOnDirectives(
+export function ProvidedRequiredArgumentsOnDirectivesRule(
   context: ValidationContext | SDLValidationContext,
 ): ASTVisitor {
   const requiredArgsMap = Object.create(null);

src/validation/specifiedRules.js

@@ -60,7 +60,7 @@ import { UniqueDirectivesPerLocationRule } from './rules/UniqueDirectivesPerLoca
 // Spec Section: "Argument Names"
 import {
   KnownArgumentNamesRule,
-  KnownArgumentNamesOnDirectives,
+  KnownArgumentNamesOnDirectivesRule,
 } from './rules/KnownArgumentNamesRule';
 
 // Spec Section: "Argument Uniqueness"
@@ -72,7 +72,7 @@ import { ValuesOfCorrectTypeRule } from './rules/ValuesOfCorrectTypeRule';
 // Spec Section: "Argument Optionality"
 import {
   ProvidedRequiredArgumentsRule,
-  ProvidedRequiredArgumentsOnDirectives,
+  ProvidedRequiredArgumentsOnDirectivesRule,
 } from './rules/ProvidedRequiredArgumentsRule';
 
 // Spec Section: "All Variable Usages Are Allowed"
@@ -142,8 +142,8 @@ export const specifiedSDLRules = Object.freeze([
   KnownDirectivesRule,
   UniqueDirectivesPerLocationRule,
   PossibleTypeExtensionsRule,
-  KnownArgumentNamesOnDirectives,
+  KnownArgumentNamesOnDirectivesRule,
   UniqueArgumentNamesRule,
   UniqueInputFieldNamesRule,
-  ProvidedRequiredArgumentsOnDirectives,
+  ProvidedRequiredArgumentsOnDirectivesRule,
 ]);