Fixed unit tests after upgrading to 10.0 and added one to reproduce #168

Author: oliemansm

src/test/groovy/com/coxautodev/graphql/tools/MethodFieldResolverDataFetcherSpec.groovy

@@ -8,6 +8,7 @@ import graphql.execution.ExecutionId
 import graphql.execution.ExecutionStrategy
 import graphql.execution.ExecutionStrategyParameters
 import graphql.execution.instrumentation.SimpleInstrumentation
+import graphql.language.BooleanValue
 import graphql.language.FieldDefinition
 import graphql.language.InputValueDefinition
 import graphql.language.NonNullType
@@ -37,7 +38,7 @@ class MethodFieldResolverDataFetcherSpec extends Specification {
 
     def "data fetcher throws exception if resolver has too few arguments"() {
         when:
-            createFetcher("active", [new InputValueDefinition("doesNotExist")], new GraphQLQueryResolver() {
+            createFetcher("active", [new InputValueDefinition("doesNotExist", new TypeName("Boolean"))], new GraphQLQueryResolver() {
                 boolean active() { true }
             })
 
@@ -143,7 +144,7 @@ class MethodFieldResolverDataFetcherSpec extends Specification {
     def "data fetcher marshalls input object if required"() {
         setup:
             def name = "correct name"
-            def resolver = createFetcher("active", [new InputValueDefinition("input")], new GraphQLQueryResolver() {
+            def resolver = createFetcher("active", [new InputValueDefinition("input", new TypeName("InputClass"))], new GraphQLQueryResolver() {
                 boolean active(InputClass input) {
                     input instanceof InputClass && input.name == name
                 }
@@ -156,7 +157,7 @@ class MethodFieldResolverDataFetcherSpec extends Specification {
     def "data fetcher doesn't marshall input object if not required"() {
         setup:
             def name = "correct name"
-            def resolver = createFetcher("active", [new InputValueDefinition("input")], new GraphQLQueryResolver() {
+            def resolver = createFetcher("active", [new InputValueDefinition("input", new TypeName("Map"))], new GraphQLQueryResolver() {
                 boolean active(Map input) {
                     input instanceof Map && input.name == name
                 }

src/test/groovy/com/coxautodev/graphql/tools/SchemaParserBuilderSpec.groovy

@@ -21,7 +21,7 @@ class SchemaParserBuilderSpec extends Specification {
 
         where:
             schema | error
-            "invalid"                   | "0,1:7='invalid'"
-            "type Query {\ninvalid!\n}" | "4,21:21='!'"
+            "invalid"                   | "0,0:6='invalid'"
+            "type Query {\ninvalid!\n}" | "4,20:20='!'"
     }
 }

src/test/groovy/com/coxautodev/graphql/tools/SchemaParserSpec.groovy

@@ -25,15 +25,17 @@ class SchemaParserSpec extends Specification {
 
     def "builder throws FileNotFound exception when file is missing"() {
         when:
-            builder.file("/404")
+            builder.file("/404").build()
 
         then:
             thrown(FileNotFoundException)
     }
 
     def "builder doesn't throw FileNotFound exception when file is present"() {
         when:
-            builder.file("test.graphqls")
+            SchemaParser.newParser().file("test.graphqls")
+                    .resolvers(new GraphQLQueryResolver() { String getId() { "1" }})
+                    .build()
 
         then:
             noExceptionThrown()
@@ -296,8 +298,75 @@ class SchemaParserSpec extends Specification {
             sourceLocation.column == 5
             sourceLocation.sourceName == "test.graphqls"
     }
+
+    def "support enum types if only used as input type"() {
+        when:
+            SchemaParser.newParser().schemaString('''\
+                    type Query { test: Boolean }
+                    
+                    type Mutation {
+                        save(input: SaveInput!): Boolean
+                    }
+                    
+                    input SaveInput {
+                        type: EnumType!
+                    }
+                    
+                    enum EnumType {
+                        TEST
+                    }
+                '''.stripIndent())
+                    .resolvers(new GraphQLMutationResolver() {
+                        boolean save(SaveInput input) { false }
+
+                        class SaveInput {
+                            EnumType type;
+                        }
+
+                    }, new GraphQLQueryResolver() {
+                        boolean test() { false }
+                    })
+                    .dictionary(EnumType.class)
+                    .build()
+                    .makeExecutableSchema()
+
+        then:
+            noExceptionThrown()
+    }
+
+    def "support enum types if only used in input Map"() {
+        when:
+            SchemaParser.newParser().schemaString('''\
+                    type Query { test: Boolean }
+                    
+                    type Mutation {
+                        save(input: SaveInput!): Boolean
+                    }
+                    
+                    input SaveInput {
+                        type: EnumType!
+                    }
+                    
+                    enum EnumType {
+                        TEST
+                    }
+                '''.stripIndent())
+                    .resolvers(new GraphQLMutationResolver() {
+                boolean save(Map input) { false }
+            }, new GraphQLQueryResolver() {
+                boolean test() { false }
+            })
+                    .dictionary(EnumType.class)
+                    .build()
+                    .makeExecutableSchema()
+
+        then:
+            noExceptionThrown()
+    }
 }
 
+enum EnumType { TEST }
+
 class QueryWithIdResolver implements GraphQLQueryResolver {
     String getId() { null }
 }