Use else branch to report second error

graphql · Nov 25, 2020 · 15eded0 · 15eded0
1 parent 5b39acd
commit 15eded0
Showing 1 changed file with 14 additions and 0 deletions.
diff --git a/src/validation/rules/SingleFieldSubscriptionsRule.js b/src/validation/rules/SingleFieldSubscriptionsRule.js
@@ -26,6 +26,20 @@ export function SingleFieldSubscriptionsRule(
               node.selectionSet.selections.slice(1),
             ),
           );
+        } else {
+          const selection = node.selectionSet.selections[0];
+          const fieldName = selection.name.value;
+          // fieldName represents an introspection field if it starts with `__`
+          if (fieldName[0] === '_' && fieldName[1] === '_') {
+            context.reportError(
+              new GraphQLError(
+                node.name
+                  ? `Subscription "${node.name.value}" must not select an introspection top level field.`
+                  : 'Anonymous Subscription must not select an introspection top level field.',
+                node.selectionSet.selections,
+              ),
+            );
+          }
         }
         const selection = node.selectionSet.selections[0];
         const fieldName = selection.name.value;