graphql-java#414 added schema validation in subscriptions

bbakerman · andimarek · commit 959174e32b61 · 2017-05-14T12:30:28.000+02:00
diff --git a/src/main/java/graphql/schema/GraphQLSchema.java b/src/main/java/graphql/schema/GraphQLSchema.java
@@ -81,6 +81,10 @@ public boolean isSupportingMutations() {
         return mutationType != null;
     }
 
+    public boolean isSupportingSubscriptions() {
+        return subscriptionType != null;
+    }
+
     public static Builder newSchema() {
         return new Builder();
     }
diff --git a/src/main/java/graphql/schema/validation/SchemaValidator.java b/src/main/java/graphql/schema/validation/SchemaValidator.java
@@ -32,6 +32,9 @@ public Set<SchemaValidationError> validateSchema(GraphQLSchema schema) {
         if (schema.isSupportingMutations()) {
             traverse(schema.getMutationType(), rules, validationErrorCollector);
         }
+        if (schema.isSupportingSubscriptions()) {
+            traverse(schema.getSubscriptionType(), rules, validationErrorCollector);
+        }
         return validationErrorCollector.getErrors();
     }
 

Original file line number	Diff line number	Diff line change
`@@ -81,6 +81,10 @@ public boolean isSupportingMutations() {`
`81`	`81`	`return mutationType != null;`
`82`	`82`	`}`
`83`	`83`
	`84`	`+ public boolean isSupportingSubscriptions() {`
	`85`	`+ return subscriptionType != null;`
	`86`	`+ }`
	`87`	`+`
`84`	`88`	`public static Builder newSchema() {`
`85`	`89`	`return new Builder();`
`86`	`90`	`}`
Original file line number	Diff line number	Diff line change
`@@ -32,6 +32,9 @@ public Set<SchemaValidationError> validateSchema(GraphQLSchema schema) {`
`32`	`32`	`if (schema.isSupportingMutations()) {`
`33`	`33`	`traverse(schema.getMutationType(), rules, validationErrorCollector);`
`34`	`34`	`}`
	`35`	`+ if (schema.isSupportingSubscriptions()) {`
	`36`	`+ traverse(schema.getSubscriptionType(), rules, validationErrorCollector);`
	`37`	`+ }`
`35`	`38`	`return validationErrorCollector.getErrors();`
`36`	`39`	`}`
`37`	`40`