Use a singleton PropertyFetcher by default - now with SchemaGeneratorHelper that does NOT put in an entry for defaulted values- DFF tweaked

Author: bbakerman

src/main/java/graphql/schema/DataFetcherFactories.java

@@ -20,7 +20,17 @@ public class DataFetcherFactories {
      * @return a data fetcher factory that always returns the provided data fetcher
      */
     public static <T> DataFetcherFactory<T> useDataFetcher(DataFetcher<T> dataFetcher) {
-        return fieldDefinition -> dataFetcher;
+        return new DataFetcherFactory<T>() {
+            @Override
+            public DataFetcher<T> get(DataFetcherFactoryEnvironment environment) {
+                return dataFetcher;
+            }
+
+            @Override
+            public DataFetcher<T> getViaField(GraphQLFieldDefinition fieldDefinition) {
+                return dataFetcher;
+            }
+        };
     }
 
     /**

src/test/groovy/graphql/schema/DataFetcherFactoriesTest.groovy

@@ -37,4 +37,14 @@ class DataFetcherFactoriesTest extends Specification {
         then:
         value == "goodbye"
     }
+
+    def "will use given df via field"() {
+        def fetcherFactory = DataFetcherFactories.useDataFetcher(pojoDF)
+
+        when:
+        def value = fetcherFactory.getViaField(null).get(null)
+
+        then:
+        value == "goodbye"
+    }
 }