fix(selection-set-depth): fix TypeError when suggestion tried to apply to fragment that located in different file (#1110)

* fix(selection-set-depth): fix `TypeError` when suggestion tried to apply to fragment that located in different file

* Update packages/plugin/tests/selection-set-depth.spec.ts

Author: dimaMachina

.changeset/four-trains-glow.md

@@ -0,0 +1,5 @@
+---
+'@graphql-eslint/eslint-plugin': patch
+---
+
+fix(selection-set-depth): fix `TypeError` when suggestion tried to apply to fragment that located in different file

packages/plugin/src/rules/selection-set-depth.ts

@@ -123,6 +123,9 @@ const rule: GraphQLESLintRule<[SelectionSetDepthRuleConfig]> = {
                       const token = (ancestors[0] as AST.Program).tokens.find(
                         token => token.loc.start.line === line && token.loc.start.column === column - 1
                       );
+                      if (!token) {
+                        return null;
+                      }
                       const sourceCode = context.getSourceCode();
                       const foundNode = sourceCode.getNodeByRangeIndex(token.range[0]) as any;
                       const parentNode = foundNode.parent.parent;

packages/plugin/tests/__snapshots__/selection-set-depth.spec.md

@@ -66,10 +66,35 @@ exports[`Invalid #2 1`] = `
     5 |         }
 `;
 
+exports[`suggestions should not throw error when fragment is located in different file 1`] = `
+#### ⌨️ Code
+
+      1 |         {
+      2 |           viewer {
+      3 |             albums {
+      4 |               ...AlbumFields
+      5 |             }
+      6 |           }
+      7 |         }
+
+#### ⚙️ Options
+
+    {
+      "maxDepth": 2
+    }
+
+#### ❌ Error
+
+      4 |               ...AlbumFields
+    > 5 |             }
+        |              ^ '' exceeds maximum operation depth of 2
+      6 |           }
+`;
+
 exports[`suggestions should work with inline fragments 1`] = `
 #### ⌨️ Code
 
-      1 |         query {
+      1 |         {
       2 |           viewer {
       3 |             albums {
       4 |               ... on Album {
@@ -94,7 +119,7 @@ exports[`suggestions should work with inline fragments 1`] = `
 
 #### 💡 Suggestion: Remove selections
 
-    1 |         query {
+    1 |         {
     2 |           viewer {
     3 |             
     4 |           }

packages/plugin/tests/selection-set-depth.spec.ts

@@ -14,7 +14,7 @@ ruleTester.runGraphQLTests<[SelectionSetDepthRuleConfig]>('selection-set-depth',
     {
       options: [{ maxDepth: 2 }],
       code: `
-        query {
+        {
           viewer { # Level 0
             albums { # Level 1
               title # Level 2
@@ -84,7 +84,7 @@ ruleTester.runGraphQLTests<[SelectionSetDepthRuleConfig]>('selection-set-depth',
       options: [{ maxDepth: 1 }],
       errors: [{ message: "'' exceeds maximum operation depth of 1" }],
       code: /* GraphQL */ `
-        query {
+        {
           viewer {
             albums {
               ... on Album {
@@ -95,5 +95,29 @@ ruleTester.runGraphQLTests<[SelectionSetDepthRuleConfig]>('selection-set-depth',
         }
       `,
     },
+    {
+      name: 'suggestions should not throw error when fragment is located in different file',
+      parserOptions: {
+        operations: /* GraphQL */ `
+          fragment AlbumFields on Album {
+            id
+            modifier {
+              date
+            }
+          }
+        `,
+      },
+      options: [{ maxDepth: 2 }],
+      errors: [{ message: "'' exceeds maximum operation depth of 2" }],
+      code: /* GraphQL */ `
+        {
+          viewer {
+            albums {
+              ...AlbumFields
+            }
+          }
+        }
+      `,
+    },
   ],
 });