added tests for eslint directives

fixes for some other tests files

Author: dotansimha

packages/plugin/tests/eslint-directives.spec.ts

@@ -0,0 +1,74 @@
+import { GraphQLRuleTester } from '../src/testkit';
+import rule from '../src/rules/no-anonymous-operations';
+
+const ruleTester = new GraphQLRuleTester();
+
+ruleTester.runGraphQLTests('test-directives', rule, {
+  valid: [
+    {
+      code: /* GraphQL */ `
+        # eslint-disable-next-line
+        query {
+          a
+        }
+      `,
+    },
+    {
+      code: /* GraphQL */ `
+        # eslint-disable-next-line test-directives
+        query {
+          a
+        }
+      `,
+    },
+    {
+      code: `
+        query { # eslint-disable-line test-directives
+          a
+        }
+      `,
+    },
+    {
+      code: `
+        query { # eslint-disable-line
+          a
+        }
+      `,
+    },
+    {
+      code: /* GraphQL */ `
+        # eslint-disable
+        query {
+          a
+        }
+      `,
+    },
+    {
+      code: /* GraphQL */ `
+        # eslint-disable
+        query {
+          a
+        }
+      `,
+    },
+  ],
+  invalid: [
+    {
+      errors: 2,
+      code: /* GraphQL */ `
+        # eslint-disable-next-line non-existing-rule
+        query {
+          a
+        }
+      `,
+    },
+    {
+      code: /* GraphQL */ `
+        query {
+          a
+        }
+      `,
+      errors: 1,
+    },
+  ],
+});

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

@@ -3,7 +3,7 @@ import rule from '../src/rules/input-name';
 
 const ruleTester = new GraphQLRuleTester();
 
-ruleTester.run('input-name', rule, {
+ruleTester.runGraphQLTests('input-name', rule, {
   valid: [
     {
       code: 'type Mutation { SetMessage(input: SetMessageInput): String }',

packages/plugin/tests/naming-convention.spec.ts

@@ -3,7 +3,7 @@ import rule from '../src/rules/naming-convention';
 
 const ruleTester = new GraphQLRuleTester();
 
-ruleTester.run('naming-convention', rule, {
+ruleTester.runGraphQLTests('naming-convention', rule, {
   valid: [
     {
       code: `query GetUser($userId:      ID!) {
@@ -46,7 +46,7 @@ ruleTester.run('naming-convention', rule, {
         {
           leadingUnderscore: 'allow',
           trailingUnderscore: 'allow',
-          TypeDefinition: 'PascalCase',
+          ObjectTypeDefinition: 'PascalCase',
           FieldDefinition: 'camelCase',
         },
       ],

packages/plugin/tests/no-anonymous-operations.spec.ts

@@ -0,0 +1,17 @@
+import { GraphQLRuleTester } from '../src/testkit';
+import rule from '../src/rules/no-anonymous-operations';
+
+const ruleTester = new GraphQLRuleTester();
+
+ruleTester.runGraphQLTests('no-anonymous-operations', rule, {
+  valid: [
+    { code: `query myQuery { a }` },
+    { code: `mutation doSomething { a }` },
+    { code: `subscription myData { a }` },
+  ],
+  invalid: [
+    { code: `query { a }`, errors: 1 },
+    { code: `mutation { a }`, errors: 1 },
+    { code: `subscription { a }`, errors: 1 },
+  ],
+});

packages/plugin/tests/prettier.spec.ts

@@ -3,7 +3,7 @@ import rule from '../src/rules/prettier';
 
 const ruleTester = new GraphQLRuleTester();
 
-ruleTester.run('prettier', rule, {
+ruleTester.runGraphQLTests('prettier', rule, {
   valid: ['scalar Test\n'],
   invalid: [
     {

packages/plugin/tests/require-deprecation-reason.spec.ts

@@ -3,7 +3,7 @@ import rule from '../src/rules/require-deprecation-reason';
 
 const ruleTester = new GraphQLRuleTester();
 
-ruleTester.run('require-deprecation-reason', rule, {
+ruleTester.runGraphQLTests('require-deprecation-reason', rule, {
   valid: [
     {
       code: /* GraphQL */ `

packages/plugin/tests/require-description.spec.ts

@@ -4,7 +4,7 @@ import { Kind } from 'graphql';
 
 const ruleTester = new GraphQLRuleTester();
 
-ruleTester.run('require-description', rule, {
+ruleTester.runGraphQLTests('require-description', rule, {
   valid: [
     {
       code: /* GraphQL */ `