add test for known-fragment-names rule with import nested fragments…

… + update `graphql-tools/import` package (#665) * add failing test false positive case for `known-fragment-names` rule * chore: update deps
dimaMachina · Nov 12, 2021 · 1914d6a · 1914d6a
1 parent 70136fd
commit 1914d6a
Show file tree

Hide file tree

Showing 12 changed files with 379 additions and 327 deletions.
diff --git a/.changeset/short-dolphins-brush.md b/.changeset/short-dolphins-brush.md
@@ -0,0 +1,5 @@
+---
+'@graphql-eslint/eslint-plugin': minor
+---
+
+fix false positive case in `known-fragment-names` when import nested fragment
diff --git a/examples/basic/package.json b/examples/basic/package.json
@@ -13,6 +13,6 @@
   },
   "devDependencies": {
     "@graphql-eslint/eslint-plugin": "2.4.1",
-    "eslint": "8.1.0"
+    "eslint": "8.2.0"
   }
 }
diff --git a/examples/code-file/package.json b/examples/code-file/package.json
@@ -13,6 +13,6 @@
   },
   "devDependencies": {
     "@graphql-eslint/eslint-plugin": "2.4.1",
-    "eslint": "8.1.0"
+    "eslint": "8.2.0"
   }
 }
diff --git a/examples/graphql-config-code-file/package.json b/examples/graphql-config-code-file/package.json
@@ -10,10 +10,10 @@
   },
   "dependencies": {
     "graphql": "16.0.1",
-    "graphql-tag": "^2.12.5"
+    "graphql-tag": "2.12.6"
   },
   "devDependencies": {
     "@graphql-eslint/eslint-plugin": "2.4.1",
-    "eslint": "8.1.0"
+    "eslint": "8.2.0"
   }
 }
diff --git a/examples/graphql-config/package.json b/examples/graphql-config/package.json
@@ -13,6 +13,6 @@
   },
   "devDependencies": {
     "@graphql-eslint/eslint-plugin": "2.4.1",
-    "eslint": "8.1.0"
+    "eslint": "8.2.0"
   }
 }
diff --git a/examples/prettier/package.json b/examples/prettier/package.json
@@ -13,7 +13,7 @@
   },
   "devDependencies": {
     "@graphql-eslint/eslint-plugin": "2.4.1",
-    "eslint": "8.1.0",
+    "eslint": "8.2.0",
     "eslint-config-prettier": "8.3.0",
     "eslint-plugin-prettier": "4.0.0",
     "prettier": "2.4.1"

diff --git a/packages/plugin/package.json b/packages/plugin/package.json
@@ -27,13 +27,13 @@
   },
   "dependencies": {
     "@babel/code-frame": "7.16.0",
-    "@graphql-tools/code-file-loader": "^7.0.2",
-    "@graphql-tools/graphql-tag-pluck": "^7.0.2",
-    "@graphql-tools/import": "^6.3.1",
-    "@graphql-tools/utils": "^8.0.2",
-    "graphql-config": "^4.0.1",
+    "@graphql-tools/code-file-loader": "7.2.2",
+    "@graphql-tools/graphql-tag-pluck": "7.1.3",
+    "@graphql-tools/import": "6.6.1",
+    "@graphql-tools/utils": "8.5.3",
+    "graphql-config": "4.1.0",
     "graphql-depth-limit": "1.1.0",
-    "lodash.lowercase": "^4.3.0"
+    "lodash.lowercase": "4.3.0"
   },
   "devDependencies": {
     "@types/babel__code-frame": "7.0.3",

diff --git a/packages/plugin/tests/known-fragment-names.spec.ts b/packages/plugin/tests/known-fragment-names.spec.ts
@@ -1,10 +1,9 @@
 import { join } from 'path';
-import { GraphQLRuleTester } from '../src';
-import { GRAPHQL_JS_VALIDATIONS } from '../src/rules/graphql-js-validation';
+import { GraphQLRuleTester, rules } from '../src';
 
 const ruleTester = new GraphQLRuleTester();
 
-ruleTester.runGraphQLTests('known-fragment-names', GRAPHQL_JS_VALIDATIONS['known-fragment-names'], {
+ruleTester.runGraphQLTests('known-fragment-names', rules['known-fragment-names'], {
   valid: [
     {
       filename: join(__dirname, 'mocks/user.graphql'),

diff --git a/packages/plugin/tests/mocks/user-fields-with-nested-fragment.gql b/packages/plugin/tests/mocks/user-fields-with-nested-fragment.gql
@@ -0,0 +1,24 @@
+fragment UserFields on User {
+  ...AnotherUserFields
+  posts {
+    ...PostFields
+  }
+}
+
+fragment AnotherUserFields on User {
+  firstName
+}
+
+fragment PostFields on Post {
+  id
+  ...AnotherPostFields
+}
+
+fragment AnotherPostFields on Post {
+  title
+  ...YetAnotherPostFields
+}
+
+fragment YetAnotherPostFields on Post {
+  content
+}
diff --git a/packages/plugin/tests/mocks/user.graphql b/packages/plugin/tests/mocks/user.graphql
@@ -1,4 +1,4 @@
-#import "./user-fields.graphql"
+#import './user-fields-with-nested-fragment.gql'
 
 query User {
   user {

diff --git a/packages/plugin/tests/no-unused-fragments.spec.ts b/packages/plugin/tests/no-unused-fragments.spec.ts
@@ -7,11 +7,14 @@ const ruleTester = new GraphQLRuleTester();
 ruleTester.runGraphQLTests('no-unused-fragments', GRAPHQL_JS_VALIDATIONS['no-unused-fragments'], {
   valid: [
     {
-      filename: join(__dirname, 'mocks/user-fields.graphql'),
-      code: ruleTester.fromMockFile('user-fields.graphql'),
+      filename: join(__dirname, 'mocks/user-fields-with-nested-fragment.gql'),
+      code: ruleTester.fromMockFile('user-fields-with-nested-fragment.gql'),
       parserOptions: {
         schema: join(__dirname, 'mocks/user-schema.graphql'),
-        operations: [join(__dirname, 'mocks/user-fields.graphql'), join(__dirname, 'mocks/user.graphql')],
+        operations: [
+          join(__dirname, 'mocks/user-fields-with-nested-fragment.gql'),
+          join(__dirname, 'mocks/user.graphql'),
+        ],
       },
     },
     {