input-name: Ignore non-Mutation fields. (#156)

* input-name: Ignore non-Mutation fields.

* add changeset

Author: henryqdineen

.changeset/wise-paws-speak.md

@@ -0,0 +1,5 @@
+---
+'@graphql-eslint/eslint-plugin': patch
+---
+
+Fix a bug in the input-name rule to make sure it only checks fields on the Mutation type

packages/plugin/src/rules/input-name.ts

@@ -30,9 +30,15 @@ const rule: GraphQLESLintRule<InputNameRuleConfig> = {
     ],
   },
   create(context) {
+    const isMutationType = node => {
+      return (
+        (node.type === 'ObjectTypeDefinition' || node.type === 'ObjectTypeExtension') && node.name.value === 'Mutation'
+      );
+    };
+
     const listeners = {
       'FieldDefinition > InputValueDefinition': node => {
-        if (node.name.value !== 'input') {
+        if (node.name.value !== 'input' && isMutationType(node.parent.parent)) {
           context.report({
             node: node.name,
             message: `Input "${node.name.value}" should be called "input"`,
@@ -50,10 +56,13 @@ const rule: GraphQLESLintRule<InputNameRuleConfig> = {
           return currentNode;
         };
 
-        const mutationName = `${findInputType(node).parent.name.value}Input`;
+        const inputValueNode = findInputType(node);
+        if (isMutationType(inputValueNode.parent.parent)) {
+          const mutationName = `${inputValueNode.parent.name.value}Input`;
 
-        if (node.name.value !== mutationName) {
-          context.report({ node, message: `InputType "${node.name.value}" name should be "${mutationName}"` });
+          if (node.name.value !== mutationName) {
+            context.report({ node, message: `InputType "${node.name.value}" name should be "${mutationName}"` });
+          }
         }
       };
     }

packages/plugin/tests/input-name.spec.ts

@@ -23,6 +23,8 @@ ruleTester.runGraphQLTests('input-name', rule, {
       options: [{ checkInputType: true }],
     },
     'type Mutation { CreateMessage(input: String): String }',
+    'extend type Mutation { CreateMessage(input: String): String }',
+    'type Query { message(id: ID): Message }',
   ],
   invalid: [
     {
@@ -39,7 +41,7 @@ ruleTester.runGraphQLTests('input-name', rule, {
       errors: [{ message: 'InputType "String" name should be "SetMessageInput"' }],
     },
     {
-      code: 'type Mutaton { SetMessage(hello: SetMessageInput): String }',
+      code: 'type Mutation { SetMessage(hello: SetMessageInput): String }',
       options: [{ checkInputType: true }],
       errors: [{ message: 'Input "hello" should be called "input"' }],
     },