fixes

Author: dimaMachina

packages/plugin/tests/__snapshots__/require-id-when-available.spec.md

@@ -140,6 +140,36 @@ exports[`should report an error with union 1`] = `
     7 |   }
 `;
 
+exports[`should report an error with union and fragment spread 1`] = `
+#### ⌨️ Code
+
+      1 |         {
+      2 |           userOrPost {
+      3 |             ... on User {
+      4 |               ...UserFields
+      5 |             }
+      6 |           }
+      7 |         }
+
+#### ❌ Error
+
+      1 |         {
+    > 2 |           userOrPost {
+        |                      ^ Field \`userOrPost.id\` must be selected when it's available on a type.
+    Include it in your selection set or add to used fragment \`UserFields\`.
+      3 |             ... on User {
+
+#### 💡 Suggestion: Add \`id\` selection
+
+    1 |         {
+    2 |           userOrPost {
+    3 |             ... on User {
+    4 |               id ...UserFields
+    5 |             }
+    6 |           }
+    7 |         }
+`;
+
 exports[`support multiple id field names 1`] = `
 #### ⌨️ Code
 

packages/plugin/tests/require-id-when-available.spec.ts

@@ -379,5 +379,26 @@ ruleTester.runGraphQLTests<RuleOptions, true>('require-id-when-available', rule,
         documents: DOCUMENT_WITH_UNION,
       },
     },
+    {
+      name: 'should report an error with union and fragment spread',
+      errors: [MESSAGE_ID],
+      code: /* GraphQL */ `
+        {
+          userOrPost {
+            ... on User {
+              ...UserFields
+            }
+          }
+        }
+      `,
+      parserOptions: {
+        schema: USER_POST_SCHEMA,
+        documents: /* GraphQL */ `
+          fragment UserFields on User {
+            name
+          }
+        `,
+      },
+    },
   ],
 });