Fixing double 'Rule' suffix that was accidentally introduced in… (#2407)

Big thanks to @trevor-scheer for reporting it here:
https://github.com/graphql/graphql-js/pull/2402/files#r371545963

src/index.d.ts

@@ -327,12 +327,12 @@ export {
   VariablesAreInputTypesRule,
   VariablesInAllowedPositionRule,
   // SDL-specific validation rules
-  LoneSchemaDefinitionRuleRule,
-  UniqueOperationTypesRuleRule,
-  UniqueTypeNamesRuleRule,
-  UniqueEnumValueNamesRuleRule,
+  LoneSchemaDefinitionRule,
+  UniqueOperationTypesRule,
+  UniqueTypeNamesRule,
+  UniqueEnumValueNamesRule,
   UniqueFieldDefinitionNamesRule,
-  UniqueDirectiveNamesRuleRule,
+  UniqueDirectiveNamesRule,
   PossibleTypeExtensionsRule,
   ValidationRule,
 } from './validation/index';

src/index.js

@@ -325,12 +325,12 @@ export {
   VariablesAreInputTypesRule,
   VariablesInAllowedPositionRule,
   // SDL-specific validation rules
-  LoneSchemaDefinitionRuleRule,
-  UniqueOperationTypesRuleRule,
-  UniqueTypeNamesRuleRule,
-  UniqueEnumValueNamesRuleRule,
+  LoneSchemaDefinitionRule,
+  UniqueOperationTypesRule,
+  UniqueTypeNamesRule,
+  UniqueEnumValueNamesRule,
   UniqueFieldDefinitionNamesRule,
-  UniqueDirectiveNamesRuleRule,
+  UniqueDirectiveNamesRule,
   PossibleTypeExtensionsRule,
 } from './validation/index';
 

src/validation/__tests__/LoneSchemaDefinitionRule-test.js

@@ -4,16 +4,12 @@ import { describe, it } from 'mocha';
 
 import { buildSchema } from '../../utilities/buildASTSchema';
 
-import { LoneSchemaDefinitionRuleRule } from '../rules/LoneSchemaDefinitionRule';
+import { LoneSchemaDefinitionRule } from '../rules/LoneSchemaDefinitionRule';
 
 import { expectSDLValidationErrors } from './harness';
 
 function expectSDLErrors(sdlStr, schema) {
-  return expectSDLValidationErrors(
-    schema,
-    LoneSchemaDefinitionRuleRule,
-    sdlStr,
-  );
+  return expectSDLValidationErrors(schema, LoneSchemaDefinitionRule, sdlStr);
 }
 
 function expectValidSDL(sdlStr, schema) {

src/validation/__tests__/UniqueDirectiveNamesRule-test.js

@@ -4,16 +4,12 @@ import { describe, it } from 'mocha';
 
 import { buildSchema } from '../../utilities/buildASTSchema';
 
-import { UniqueDirectiveNamesRuleRule } from '../rules/UniqueDirectiveNamesRule';
+import { UniqueDirectiveNamesRule } from '../rules/UniqueDirectiveNamesRule';
 
 import { expectSDLValidationErrors } from './harness';
 
 function expectSDLErrors(sdlStr, schema) {
-  return expectSDLValidationErrors(
-    schema,
-    UniqueDirectiveNamesRuleRule,
-    sdlStr,
-  );
+  return expectSDLValidationErrors(schema, UniqueDirectiveNamesRule, sdlStr);
 }
 
 function expectValidSDL(sdlStr, schema) {

src/validation/__tests__/UniqueEnumValueNamesRule-test.js

@@ -4,16 +4,12 @@ import { describe, it } from 'mocha';
 
 import { buildSchema } from '../../utilities/buildASTSchema';
 
-import { UniqueEnumValueNamesRuleRule } from '../rules/UniqueEnumValueNamesRule';
+import { UniqueEnumValueNamesRule } from '../rules/UniqueEnumValueNamesRule';
 
 import { expectSDLValidationErrors } from './harness';
 
 function expectSDLErrors(sdlStr, schema) {
-  return expectSDLValidationErrors(
-    schema,
-    UniqueEnumValueNamesRuleRule,
-    sdlStr,
-  );
+  return expectSDLValidationErrors(schema, UniqueEnumValueNamesRule, sdlStr);
 }
 
 function expectValidSDL(sdlStr, schema) {

src/validation/__tests__/UniqueOperationTypesRule-test.js

@@ -4,16 +4,12 @@ import { describe, it } from 'mocha';
 
 import { buildSchema } from '../../utilities/buildASTSchema';
 
-import { UniqueOperationTypesRuleRule } from '../rules/UniqueOperationTypesRule';
+import { UniqueOperationTypesRule } from '../rules/UniqueOperationTypesRule';
 
 import { expectSDLValidationErrors } from './harness';
 
 function expectSDLErrors(sdlStr, schema) {
-  return expectSDLValidationErrors(
-    schema,
-    UniqueOperationTypesRuleRule,
-    sdlStr,
-  );
+  return expectSDLValidationErrors(schema, UniqueOperationTypesRule, sdlStr);
 }
 
 function expectValidSDL(sdlStr, schema) {

src/validation/__tests__/UniqueTypeNamesRule-test.js

@@ -4,12 +4,12 @@ import { describe, it } from 'mocha';
 
 import { buildSchema } from '../../utilities/buildASTSchema';
 
-import { UniqueTypeNamesRuleRule } from '../rules/UniqueTypeNamesRule';
+import { UniqueTypeNamesRule } from '../rules/UniqueTypeNamesRule';
 
 import { expectSDLValidationErrors } from './harness';
 
 function expectSDLErrors(sdlStr, schema) {
-  return expectSDLValidationErrors(schema, UniqueTypeNamesRuleRule, sdlStr);
+  return expectSDLValidationErrors(schema, UniqueTypeNamesRule, sdlStr);
 }
 
 function expectValidSDL(sdlStr, schema) {

src/validation/index.d.ts

@@ -83,10 +83,10 @@ export { VariablesAreInputTypesRule } from './rules/VariablesAreInputTypesRule';
 export { VariablesInAllowedPositionRule } from './rules/VariablesInAllowedPositionRule';
 
 // SDL-specific validation rules
-export { LoneSchemaDefinitionRuleRule } from './rules/LoneSchemaDefinitionRule';
-export { UniqueOperationTypesRuleRule } from './rules/UniqueOperationTypesRule';
-export { UniqueTypeNamesRuleRule } from './rules/UniqueTypeNamesRule';
-export { UniqueEnumValueNamesRuleRule } from './rules/UniqueEnumValueNamesRule';
+export { LoneSchemaDefinitionRule } from './rules/LoneSchemaDefinitionRule';
+export { UniqueOperationTypesRule } from './rules/UniqueOperationTypesRule';
+export { UniqueTypeNamesRule } from './rules/UniqueTypeNamesRule';
+export { UniqueEnumValueNamesRule } from './rules/UniqueEnumValueNamesRule';
 export { UniqueFieldDefinitionNamesRule } from './rules/UniqueFieldDefinitionNamesRule';
-export { UniqueDirectiveNamesRuleRule } from './rules/UniqueDirectiveNamesRule';
+export { UniqueDirectiveNamesRule } from './rules/UniqueDirectiveNamesRule';
 export { PossibleTypeExtensionsRule } from './rules/PossibleTypeExtensionsRule';

src/validation/index.js

@@ -87,10 +87,10 @@ export { VariablesAreInputTypesRule } from './rules/VariablesAreInputTypesRule';
 export { VariablesInAllowedPositionRule } from './rules/VariablesInAllowedPositionRule';
 
 // SDL-specific validation rules
-export { LoneSchemaDefinitionRuleRule } from './rules/LoneSchemaDefinitionRule';
-export { UniqueOperationTypesRuleRule } from './rules/UniqueOperationTypesRule';
-export { UniqueTypeNamesRuleRule } from './rules/UniqueTypeNamesRule';
-export { UniqueEnumValueNamesRuleRule } from './rules/UniqueEnumValueNamesRule';
+export { LoneSchemaDefinitionRule } from './rules/LoneSchemaDefinitionRule';
+export { UniqueOperationTypesRule } from './rules/UniqueOperationTypesRule';
+export { UniqueTypeNamesRule } from './rules/UniqueTypeNamesRule';
+export { UniqueEnumValueNamesRule } from './rules/UniqueEnumValueNamesRule';
 export { UniqueFieldDefinitionNamesRule } from './rules/UniqueFieldDefinitionNamesRule';
-export { UniqueDirectiveNamesRuleRule } from './rules/UniqueDirectiveNamesRule';
+export { UniqueDirectiveNamesRule } from './rules/UniqueDirectiveNamesRule';
 export { PossibleTypeExtensionsRule } from './rules/PossibleTypeExtensionsRule';

src/validation/rules/LoneSchemaDefinitionRule.d.ts

@@ -6,6 +6,6 @@ import { SDLValidationContext } from '../ValidationContext';
  *
  * A GraphQL document is only valid if it contains only one schema definition.
  */
-export function LoneSchemaDefinitionRuleRule(
+export function LoneSchemaDefinitionRule(
   context: SDLValidationContext,
 ): ASTVisitor;

src/validation/rules/LoneSchemaDefinitionRule.js

@@ -10,7 +10,7 @@ import { type SDLValidationContext } from '../ValidationContext';
  *
  * A GraphQL document is only valid if it contains only one schema definition.
  */
-export function LoneSchemaDefinitionRuleRule(
+export function LoneSchemaDefinitionRule(
   context: SDLValidationContext,
 ): ASTVisitor {
   const oldSchema = context.getSchema();

src/validation/rules/UniqueDirectiveNamesRule.d.ts

@@ -6,6 +6,6 @@ import { SDLValidationContext } from '../ValidationContext';
  *
  * A GraphQL document is only valid if all defined directives have unique names.
  */
-export function UniqueDirectiveNamesRuleRule(
+export function UniqueDirectiveNamesRule(
   context: SDLValidationContext,
 ): ASTVisitor;

src/validation/rules/UniqueDirectiveNamesRule.js

@@ -10,7 +10,7 @@ import { type SDLValidationContext } from '../ValidationContext';
  *
  * A GraphQL document is only valid if all defined directives have unique names.
  */
-export function UniqueDirectiveNamesRuleRule(
+export function UniqueDirectiveNamesRule(
   context: SDLValidationContext,
 ): ASTVisitor {
   const knownDirectiveNames = Object.create(null);

src/validation/rules/UniqueEnumValueNamesRule.d.ts

@@ -6,6 +6,6 @@ import { SDLValidationContext } from '../ValidationContext';
  *
  * A GraphQL enum type is only valid if all its values are uniquely named.
  */
-export function UniqueEnumValueNamesRuleRule(
+export function UniqueEnumValueNamesRule(
   context: SDLValidationContext,
 ): ASTVisitor;

src/validation/rules/UniqueEnumValueNamesRule.js

@@ -11,7 +11,7 @@ import { type SDLValidationContext } from '../ValidationContext';
  *
  * A GraphQL enum type is only valid if all its values are uniquely named.
  */
-export function UniqueEnumValueNamesRuleRule(
+export function UniqueEnumValueNamesRule(
   context: SDLValidationContext,
 ): ASTVisitor {
   const schema = context.getSchema();

src/validation/rules/UniqueOperationTypesRule.d.ts

@@ -6,6 +6,6 @@ import { SDLValidationContext } from '../ValidationContext';
  *
  * A GraphQL document is only valid if it has only one type per operation.
  */
-export function UniqueOperationTypesRuleRule(
+export function UniqueOperationTypesRule(
   context: SDLValidationContext,
 ): ASTVisitor;

src/validation/rules/UniqueOperationTypesRule.js

@@ -10,7 +10,7 @@ import { type SDLValidationContext } from '../ValidationContext';
  *
  * A GraphQL document is only valid if it has only one type per operation.
  */
-export function UniqueOperationTypesRuleRule(
+export function UniqueOperationTypesRule(
   context: SDLValidationContext,
 ): ASTVisitor {
   const schema = context.getSchema();

src/validation/rules/UniqueTypeNamesRule.d.ts

@@ -6,6 +6,4 @@ import { SDLValidationContext } from '../ValidationContext';
  *
  * A GraphQL document is only valid if all defined types have unique names.
  */
-export function UniqueTypeNamesRuleRule(
-  context: SDLValidationContext,
-): ASTVisitor;
+export function UniqueTypeNamesRule(context: SDLValidationContext): ASTVisitor;

src/validation/rules/UniqueTypeNamesRule.js

@@ -12,9 +12,7 @@ import { type SDLValidationContext } from '../ValidationContext';
  *
  * A GraphQL document is only valid if all defined types have unique names.
  */
-export function UniqueTypeNamesRuleRule(
-  context: SDLValidationContext,
-): ASTVisitor {
+export function UniqueTypeNamesRule(context: SDLValidationContext): ASTVisitor {
   const knownTypeNames = Object.create(null);
   const schema = context.getSchema();
 

src/validation/specifiedRules.js

@@ -85,12 +85,12 @@ import { OverlappingFieldsCanBeMergedRule } from './rules/OverlappingFieldsCanBe
 import { UniqueInputFieldNamesRule } from './rules/UniqueInputFieldNamesRule';
 
 // SDL-specific validation rules
-import { LoneSchemaDefinitionRuleRule } from './rules/LoneSchemaDefinitionRule';
-import { UniqueOperationTypesRuleRule } from './rules/UniqueOperationTypesRule';
-import { UniqueTypeNamesRuleRule } from './rules/UniqueTypeNamesRule';
-import { UniqueEnumValueNamesRuleRule } from './rules/UniqueEnumValueNamesRule';
+import { LoneSchemaDefinitionRule } from './rules/LoneSchemaDefinitionRule';
+import { UniqueOperationTypesRule } from './rules/UniqueOperationTypesRule';
+import { UniqueTypeNamesRule } from './rules/UniqueTypeNamesRule';
+import { UniqueEnumValueNamesRule } from './rules/UniqueEnumValueNamesRule';
 import { UniqueFieldDefinitionNamesRule } from './rules/UniqueFieldDefinitionNamesRule';
-import { UniqueDirectiveNamesRuleRule } from './rules/UniqueDirectiveNamesRule';
+import { UniqueDirectiveNamesRule } from './rules/UniqueDirectiveNamesRule';
 import { PossibleTypeExtensionsRule } from './rules/PossibleTypeExtensionsRule';
 
 /**
@@ -132,12 +132,12 @@ export const specifiedRules = Object.freeze([
  * @internal
  */
 export const specifiedSDLRules = Object.freeze([
-  LoneSchemaDefinitionRuleRule,
-  UniqueOperationTypesRuleRule,
-  UniqueTypeNamesRuleRule,
-  UniqueEnumValueNamesRuleRule,
+  LoneSchemaDefinitionRule,
+  UniqueOperationTypesRule,
+  UniqueTypeNamesRule,
+  UniqueEnumValueNamesRule,
   UniqueFieldDefinitionNamesRule,
-  UniqueDirectiveNamesRuleRule,
+  UniqueDirectiveNamesRule,
   KnownTypeNamesRule,
   KnownDirectivesRule,
   UniqueDirectivesPerLocationRule,