Default schema description to null when not provided

Author: Tom Groves

src/main/kotlin/graphql/kickstart/tools/RootTypeInfo.kt

@@ -17,7 +17,7 @@ internal class RootTypeInfo private constructor(
         const val DEFAULT_QUERY_NAME = "Query"
         const val DEFAULT_MUTATION_NAME = "Mutation"
         const val DEFAULT_SUBSCRIPTION_NAME = "Subscription"
-        const val DEFAULT_DESCRIPTION = "A GraphQL schema provides a root type for each kind of operation."
+        val DEFAULT_DESCRIPTION: String? = null // According to the GraphQL Specification description should be a string or `null`
 
         fun fromSchemaDefinitions(definitions: List<SchemaDefinition>): RootTypeInfo {
             val queryType = definitions.lastOrNull()?.operationTypeDefinitions?.find { it.name == "query" }?.typeName

src/test/kotlin/graphql/kickstart/tools/SchemaParserTest.kt

@@ -575,7 +575,7 @@ class SchemaParserTest {
     }
 
     @Test
-    fun `parser should include schema descriptions`() {
+    fun `parser should include schema descriptions when declared`() {
         val schema = SchemaParser.newParser()
             .schemaString(
                 """
@@ -600,6 +600,31 @@ class SchemaParserTest {
         assertEquals(schema.description, "This is a schema level description")
     }
 
+    @Test
+    fun `parser should return null schema description when not declared`() {
+        val schema = SchemaParser.newParser()
+            .schemaString(
+                """
+                schema {
+                  query: SubstituteQuery
+                }
+
+                type SubstituteQuery {
+                    description: String
+                    comment: String
+                    omitted: String
+                    both: String
+                    empty: String
+                }
+                """)
+            .resolvers(object : GraphQLQueryResolver {})
+            .options(SchemaParserOptions.newOptions().allowUnimplementedResolvers(true).build())
+            .build()
+            .makeExecutableSchema()
+
+        assertNull(schema.description)
+    }
+
     enum class EnumType {
         TEST
     }