Better testing

Author: bbakerman

src/main/java/graphql/schema/idl/TypeRuntimeWiring.java

@@ -143,8 +143,8 @@ public Builder strictMode() {
         public Builder dataFetcher(String fieldName, DataFetcher dataFetcher) {
             assertNotNull(dataFetcher, () -> "you must provide a data fetcher");
             assertNotNull(fieldName, () -> "you must tell us what field");
-            if (strictMode && fieldDataFetchers.containsKey(fieldName)) {
-                throw new StrictModeWiringException(format("The field %s already has a data fetcher defined", fieldName));
+            if (strictMode) {
+                assertFieldStrictly(fieldName);
             }
             fieldDataFetchers.put(fieldName, dataFetcher);
             return this;
@@ -161,15 +161,19 @@ public Builder dataFetchers(Map<String, DataFetcher> dataFetchersMap) {
             assertNotNull(dataFetchersMap, () -> "you must provide a data fetchers map");
             if (strictMode) {
                 dataFetchersMap.forEach((fieldName, df) -> {
-                    if (fieldDataFetchers.containsKey(fieldName)) {
-                        throw new StrictModeWiringException(format("The field %s already has a data fetcher defined", fieldName));
-                    }
+                    assertFieldStrictly(fieldName);
                 });
             }
             fieldDataFetchers.putAll(dataFetchersMap);
             return this;
         }
 
+        private void assertFieldStrictly(String fieldName) {
+            if (fieldDataFetchers.containsKey(fieldName)) {
+                throw new StrictModeWiringException(format("The field %s already has a data fetcher defined", fieldName));
+            }
+        }
+
         /**
          * All fields in a type need a data fetcher of some sort and this method is called to provide the default data fetcher
          * that will be used for this type if no specific one has been provided per field.

src/test/groovy/graphql/schema/idl/TypeRuntimeWiringTest.groovy

@@ -35,7 +35,20 @@ class TypeRuntimeWiringTest extends Specification {
                 .dataFetcher("foo", DF2)
                 .build()
         then:
-        thrown(StrictModeWiringException)
+        def e = thrown(StrictModeWiringException)
+        e.message == "The field foo already has a data fetcher defined"
+    }
+
+    def "strict mode can be turned on for maps of fields"() {
+        when:
+        TypeRuntimeWiring.newTypeWiring("Foo")
+                .strictMode()
+                .dataFetcher("foo", DF1)
+                .dataFetchers(["foo": DF2])
+                .build()
+        then:
+        def e = thrown(StrictModeWiringException)
+        e.message == "The field foo already has a data fetcher defined"
     }
 
     def "strict mode can be turned on JVM wide"() {
@@ -57,7 +70,8 @@ class TypeRuntimeWiringTest extends Specification {
                 .build()
         then:
         inStrictMode
-        thrown(StrictModeWiringException)
+        def e = thrown(StrictModeWiringException)
+        e.message == "The field foo already has a data fetcher defined"
 
         when:
         TypeRuntimeWiring.setStrictModeJvmWide(false)