Fixed couple of unit tests related to kotlinFunction

Author: oliemansm

src/main/kotlin/com/coxautodev/graphql/tools/FieldResolverScanner.kt

@@ -9,6 +9,7 @@ import org.apache.commons.lang3.reflect.FieldUtils
 import org.slf4j.LoggerFactory
 import java.lang.reflect.Modifier
 import java.lang.reflect.ParameterizedType
+import java.lang.reflect.Type
 import kotlin.reflect.full.valueParameters
 import kotlin.reflect.jvm.javaType
 import kotlin.reflect.jvm.kotlinFunction
@@ -117,15 +118,31 @@ internal class FieldResolverScanner(val options: SchemaParserOptions) {
             true
         }
 
-        val methodParameterCount = method.kotlinFunction?.valueParameters?.size ?: method.parameterCount
-        val methodLastParameter = method.kotlinFunction?.valueParameters?.lastOrNull()?.type?.javaType
-                ?: method.parameterTypes.lastOrNull()
+        val methodParameterCount = getMethodParameterCount(method)
+        val methodLastParameter = getMethodLastParameter(method)
 
         val correctParameterCount = methodParameterCount == requiredCount ||
                 (methodParameterCount == (requiredCount + 1) && allowedLastArgumentTypes.contains(methodLastParameter))
         return correctParameterCount && appropriateFirstParameter
     }
 
+    private fun getMethodParameterCount(method: java.lang.reflect.Method): Int {
+        return try {
+            method.kotlinFunction?.valueParameters?.size ?: method.parameterCount
+        } catch (e: InternalError) {
+            method.parameterCount
+        }
+    }
+
+    private fun getMethodLastParameter(method: java.lang.reflect.Method): Type? {
+        return try {
+            method.kotlinFunction?.valueParameters?.lastOrNull()?.type?.javaType
+                    ?: method.parameterTypes.lastOrNull()
+        } catch (e: InternalError) {
+            method.parameterTypes.lastOrNull()
+        }
+    }
+
     private fun verifyBatchedMethodFirstArgument(firstType: JavaType?, requiredFirstParameterType: Class<*>?): Boolean {
         if (firstType == null) {
             return false

src/main/kotlin/com/coxautodev/graphql/tools/MethodFieldResolver.kt

@@ -35,7 +35,12 @@ internal class MethodFieldResolver(field: FieldDefinition, search: FieldResolver
         }
     }
 
-    private val additionalLastArgument = method.kotlinFunction?.valueParameters?.size ?: method.parameterCount == (field.inputValueDefinitions.size + getIndexOffset() + 1)
+    private val additionalLastArgument =
+            try {
+                method.kotlinFunction?.valueParameters?.size ?: method.parameterCount == (field.inputValueDefinitions.size + getIndexOffset() + 1)
+            } catch (e: InternalError) {
+                method.parameterCount == (field.inputValueDefinitions.size + getIndexOffset() + 1)
+            }
 
     override fun createDataFetcher(): DataFetcher<*> {
         val batched = isBatched(method, search)
@@ -103,7 +108,7 @@ internal class MethodFieldResolver(field: FieldDefinition, search: FieldResolver
 
     override fun scanForMatches(): List<TypeClassMatcher.PotentialMatch> {
         val batched = isBatched(method, search)
-        val unwrappedGenericType = genericType.unwrapGenericType(method.kotlinFunction?.returnType?.javaType ?: method.genericReturnType)
+        val unwrappedGenericType = genericType.unwrapGenericType(try { method.kotlinFunction?.returnType?.javaType ?: method.genericReturnType } catch (e: InternalError) { method.genericReturnType })
         val returnValueMatch = TypeClassMatcher.PotentialMatch.returnValue(field.type, unwrappedGenericType, genericType, SchemaClassScanner.ReturnValueReference(method), batched)
 
         return field.inputValueDefinitions.mapIndexed { i, inputDefinition ->
@@ -140,7 +145,7 @@ open class MethodFieldResolverDataFetcher(private val sourceResolver: SourceReso
     // Convert to reflactasm reflection
     private val methodAccess = MethodAccess.get(method.declaringClass)!!
     private val methodIndex = methodAccess.getIndex(method.name, *method.parameterTypes)
-    private val isSuspendFunction = method.kotlinFunction?.isSuspend == true
+    private val isSuspendFunction = try {method.kotlinFunction?.isSuspend == true } catch (e: InternalError) { false }
 
     private class CompareGenericWrappers {
         companion object : Comparator<GenericWrapper> {