Avoid array of Kotlin primitive types to be mapped as array of java primitives (#33)

Author: gscheibel

src/main/kotlin/com/expedia/graphql/schema/generator/SchemaGenerator.kt

@@ -41,6 +41,7 @@ import kotlin.reflect.full.declaredMemberProperties
 import kotlin.reflect.full.isSubclassOf
 import kotlin.reflect.full.superclasses
 import kotlin.reflect.full.valueParameters
+import kotlin.reflect.jvm.javaType
 import kotlin.reflect.jvm.jvmErasure
 
 @Suppress("Detekt.UnsafeCast")
@@ -138,7 +139,7 @@ internal class SchemaGenerator(
                 throw IllegalArgumentException("argument name is null or blank, $it")
             } else {
                 // Kotlin 1.3 will support contracts, until then we need to force non-null
-                args[name!!] = Parameter(it.type.jvmErasure.java, it.annotations)
+                args[name!!] = Parameter(it.type.javaType as Class<*>, it.annotations)
             }
         }
 

src/test/kotlin/com/expedia/graphql/schema/generator/SchemaGeneratorTest.kt

@@ -32,6 +32,32 @@ class SchemaGeneratorTest {
         assertEquals(1, geo?.get("query")?.get("id"))
     }
 
+    @Test
+    fun `Schema generator exposes arrays of primitive types as function arguments`() {
+        val schema = toSchema(listOf(TopLevelObjectDef(QueryWithArray())), config = testSchemaConfig)
+        val firstArgumentType = schema.queryType.getFieldDefinition("sumOf").arguments[0].type.deepName
+        assertEquals("[Int!]!", firstArgumentType)
+
+        val graphQL = GraphQL.newGraphQL(schema).build()
+        val result = graphQL.execute("{ sumOf(ints: [1, 2, 3]) }")
+        val sum = result.getData<Map<String, Int>>().values.first()
+
+        assertEquals(6, sum)
+    }
+
+    @Test
+    fun `Schema generator exposes arrays of complex types as function arguments`() {
+        val schema = toSchema(listOf(TopLevelObjectDef(QueryWithArray())), config = testSchemaConfig)
+        val firstArgumentType = schema.queryType.getFieldDefinition("sumOfComplexArray").arguments[0].type.deepName
+        assertEquals("[ComplexWrappingTypeInput!]!", firstArgumentType)
+
+        val graphQL = GraphQL.newGraphQL(schema).build()
+        val result = graphQL.execute("{sumOfComplexArray(objects: [{value: 1}, {value: 2}, {value: 3}])}")
+        val sum = result.getData<Map<String, Int>>().values.first()
+
+        assertEquals(6, sum)
+    }
+
     @Test
     fun `SchemaGenerator ignores fields and functions with @Ignore`() {
         val schema = toSchema(listOf(TopLevelObjectDef(QueryWithIgnored())), config = testSchemaConfig)
@@ -179,6 +205,11 @@ class SchemaGeneratorTest {
         fun query(@GraphQLDescription("A GraphQL value") value: Int): Geography = Geography(value, GeoType.CITY, listOf())
     }
 
+    class QueryWithArray {
+        fun sumOf(ints: Array<Int>): Int = ints.sum()
+        fun sumOfComplexArray(objects: Array<ComplexWrappingType>): Int = objects.map { it.value }.sum()
+    }
+
     class QueryWithIgnored {
         fun query(): ResultWithIgnored? = null
 
@@ -200,6 +231,8 @@ class SchemaGeneratorTest {
         fun mutation(value: Int): Boolean = value > 0
     }
 
+    data class ComplexWrappingType(val value: Int)
+
     @GraphQLDescription("A place")
     data class Geography(
         val id: Int?,