Function builder ignores parameters annotated with GraphQLIgnore (ExpediaGroup#207)

Author: sgeb

src/main/kotlin/com/expedia/graphql/generator/types/FunctionBuilder.kt

@@ -9,6 +9,7 @@ import com.expedia.graphql.generator.extensions.getName
 import com.expedia.graphql.generator.extensions.getTypeOfFirstArgument
 import com.expedia.graphql.generator.extensions.isDataFetchingEnvironment
 import com.expedia.graphql.generator.extensions.isGraphQLContext
+import com.expedia.graphql.generator.extensions.isGraphQLIgnored
 import com.expedia.graphql.generator.extensions.isInterface
 import com.expedia.graphql.generator.extensions.safeCast
 import graphql.schema.GraphQLArgument
@@ -37,6 +38,7 @@ internal class FunctionBuilder(generator: SchemaGenerator) : TypeBuilder(generat
 
         fn.valueParameters
             .filterNot { it.isGraphQLContext() }
+            .filterNot { it.isGraphQLIgnored() }
             .filterNot { it.isDataFetchingEnvironment() }
             .forEach {
                 // deprecation of arguments is currently unsupported: https://github.com/facebook/graphql/issues/197

src/test/kotlin/com/expedia/graphql/generator/types/FunctionBuilderTest.kt

@@ -3,6 +3,7 @@ package com.expedia.graphql.generator.types
 import com.expedia.graphql.annotations.GraphQLContext
 import com.expedia.graphql.annotations.GraphQLDescription
 import com.expedia.graphql.annotations.GraphQLDirective
+import com.expedia.graphql.annotations.GraphQLIgnore
 import com.expedia.graphql.execution.FunctionDataFetcher
 import graphql.Scalars
 import graphql.introspection.Introspection
@@ -52,6 +53,8 @@ internal class FunctionBuilderTest : TypeTestHelper() {
 
         fun context(@GraphQLContext context: String, string: String) = "$context and $string"
 
+        fun ignoredParameter(color: String, @GraphQLIgnore ignoreMe: String) = "$color and $ignoreMe"
+
         fun completableFuture(num: Int): CompletableFuture<Int> = CompletableFuture.completedFuture(num)
 
         fun dataFetchingEnvironment(environment: DataFetchingEnvironment): String = environment.field.name
@@ -128,6 +131,17 @@ internal class FunctionBuilderTest : TypeTestHelper() {
         assertEquals(expected = "string", actual = arg?.name)
     }
 
+    @Test
+    fun `Test ignored parameter`() {
+        val kFunction = Happy::ignoredParameter
+        val result = builder.function(kFunction)
+
+        assertTrue(result.directives.isEmpty())
+        assertEquals(expected = 1, actual = result.arguments.size)
+        val arg = result.arguments.firstOrNull()
+        assertEquals(expected = "color", actual = arg?.name)
+    }
+
     @Test
     fun `Non-abstract function`() {
         val kFunction = MyInterface::printMessage