Fix comments in defer/stream validation rules (#3781)

robrichard · web-flow · commit d23e5debf4cf · 2022-11-21T19:44:25.000+02:00
diff --git a/src/validation/__tests__/DeferStreamDirectiveOnRootFieldRule-test.ts b/src/validation/__tests__/DeferStreamDirectiveOnRootFieldRule-test.ts
@@ -160,7 +160,7 @@ describe('Validate: Defer/Stream directive on root field', () => {
     expectValid(`
       subscription {
         subscriptionField {
-          ...nestedFragment
+          ...nestedFragment @defer
         }
       }
       fragment nestedFragment on Message {
diff --git a/src/validation/rules/DeferStreamDirectiveLabelRule.ts b/src/validation/rules/DeferStreamDirectiveLabelRule.ts
@@ -11,7 +11,7 @@ import {
 import type { ValidationContext } from '../ValidationContext.js';
 
 /**
- * Stream directive on list field
+ * Defer and stream directive labels are unique
  *
  * A GraphQL document is only valid if defer and stream directives' label argument is static and unique.
  */
diff --git a/src/validation/rules/DeferStreamDirectiveOnRootFieldRule.ts b/src/validation/rules/DeferStreamDirectiveOnRootFieldRule.ts
@@ -10,7 +10,7 @@ import {
 import type { ValidationContext } from '../ValidationContext.js';
 
 /**
- * Stream directive on list field
+ * Defer and stream directives are used on valid root field
  *
  * A GraphQL document is only valid if defer directives are not used on root mutation or subscription types.
  */
diff --git a/src/validation/rules/StreamDirectiveOnListFieldRule.ts b/src/validation/rules/StreamDirectiveOnListFieldRule.ts
@@ -9,7 +9,7 @@ import { GraphQLStreamDirective } from '../../type/directives.js';
 import type { ValidationContext } from '../ValidationContext.js';
 
 /**
- * Stream directive on list field
+ * Stream directives are used on list fields
  *
  * A GraphQL document is only valid if stream directives are used on list fields.
  */

Original file line number	Diff line number	Diff line change
`@@ -160,7 +160,7 @@ describe('Validate: Defer/Stream directive on root field', () => {`
`160`	`160`	expectValid(`
`161`	`161`	`subscription {`
`162`	`162`	`subscriptionField {`
`163`		`- ...nestedFragment`
	`163`	`+ ...nestedFragment @defer`
`164`	`164`	`}`
`165`	`165`	`}`
`166`	`166`	`fragment nestedFragment on Message {`
Original file line number	Diff line number	Diff line change
`@@ -11,7 +11,7 @@ import {`
`11`	`11`	`import type { ValidationContext } from '../ValidationContext.js';`
`12`	`12`
`13`	`13`	`/**`
`14`		`- * Stream directive on list field`
	`14`	`+ * Defer and stream directive labels are unique`
`15`	`15`	`*`
`16`	`16`	`* A GraphQL document is only valid if defer and stream directives' label argument is static and unique.`
`17`	`17`	`*/`
Original file line number	Diff line number	Diff line change
`@@ -10,7 +10,7 @@ import {`
`10`	`10`	`import type { ValidationContext } from '../ValidationContext.js';`
`11`	`11`
`12`	`12`	`/**`
`13`		`- * Stream directive on list field`
	`13`	`+ * Defer and stream directives are used on valid root field`
`14`	`14`	`*`
`15`	`15`	`* A GraphQL document is only valid if defer directives are not used on root mutation or subscription types.`
`16`	`16`	`*/`
Original file line number	Diff line number	Diff line change
`@@ -9,7 +9,7 @@ import { GraphQLStreamDirective } from '../../type/directives.js';`
`9`	`9`	`import type { ValidationContext } from '../ValidationContext.js';`
`10`	`10`
`11`	`11`	`/**`
`12`		`- * Stream directive on list field`
	`12`	`+ * Stream directives are used on list fields`
`13`	`13`	`*`
`14`	`14`	`* A GraphQL document is only valid if stream directives are used on list fields.`
`15`	`15`	`*/`