#383 Added a search in FieldResolverScanner for getters where the field name needs to be converted from snake to camel case.

Author: plaurina

src/main/kotlin/graphql/kickstart/tools/resolver/FieldResolverScanner.kt

@@ -12,7 +12,9 @@ import graphql.language.FieldDefinition
 import graphql.language.TypeName
 import graphql.schema.DataFetchingEnvironment
 import org.apache.commons.lang3.ClassUtils
+import org.apache.commons.lang3.StringUtils
 import org.apache.commons.lang3.reflect.FieldUtils
+import org.apache.commons.lang3.text.WordUtils
 import org.slf4j.LoggerFactory
 import java.lang.reflect.Method
 import java.lang.reflect.Modifier
@@ -83,6 +85,7 @@ internal class FieldResolverScanner(val options: SchemaParserOptions) {
         //   2. Method that returns a boolean with "is" style getter
         //   3. Method with "get" style getter
         //   4. Method with "getField" style getter
+        //   5. Method with "get" style getter with the field name converted from snake_case to camelCased. ex: key_ops -> getKeyOps()
         return methods.find {
             it.name == name && verifyMethodArguments(it, argumentCount, search)
         } ?: methods.find {
@@ -91,6 +94,8 @@ internal class FieldResolverScanner(val options: SchemaParserOptions) {
             it.name == "get${name.capitalize()}" && verifyMethodArguments(it, argumentCount, search)
         } ?: methods.find {
             it.name == "getField${name.capitalize()}" && verifyMethodArguments(it, argumentCount, search)
+        } ?: methods.find {
+            it.name == "get${StringUtils.remove(WordUtils.capitalizeFully(name, '_'), "_")}" && verifyMethodArguments(it, argumentCount, search)
         }
     }
 

src/test/groovy/graphql/kickstart/tools/FieldResolverScannerSpec.groovy

@@ -87,6 +87,18 @@ class FieldResolverScannerSpec extends Specification {
         ((MethodFieldResolver) meta).getMethod().getReturnType() == BoatInformation.class
     }
 
+    def "scanner finds field resolver method using camelCase for snake_cased field_name"() {
+        setup:
+        def resolver = new RootResolverInfo([new CamelCaseQuery1()], options)
+
+        when:
+        def meta = scanner.findFieldResolver(new FieldDefinition("hull_type", new TypeName("HullType")), resolver)
+
+        then:
+        meta instanceof MethodFieldResolver
+        ((MethodFieldResolver) meta).getMethod().getReturnType() == HullType.class
+    }
+
     class RootQuery1 implements GraphQLQueryResolver {
         def field1() {}
     }
@@ -99,6 +111,12 @@ class FieldResolverScannerSpec extends Specification {
         def field1() {}
     }
 
+    class CamelCaseQuery1 implements GraphQLQueryResolver {
+        HullType getHullType(){}
+    }
+
+    class HullType {}
+
     class ParentPropertyQuery {
         private Integer version = 1
     }