upgrade to graphql-java 13 (#242)

* WIP upgrade to graphql java 12

* upgrade to graphql-java 13

* refactor directive wiring factory to not depend on graphql-java

* fix directive logic

* update interface/union type resolvers to use code registry

* simplify hooks interface

* add missing directive wiring tests

* Fix ktlint issue

Author: dariuszkuc

example/src/main/kotlin/com/expedia/graphql/sample/Application.kt

@@ -1,14 +1,13 @@
 package com.expedia.graphql.sample
 
-import com.expedia.graphql.DirectiveWiringHelper
 import com.expedia.graphql.SchemaGeneratorConfig
 import com.expedia.graphql.TopLevelObject
+import com.expedia.graphql.directives.KotlinDirectiveWiringFactory
 import com.expedia.graphql.execution.KotlinDataFetcherFactoryProvider
 import com.expedia.graphql.hooks.SchemaGeneratorHooks
 import com.expedia.graphql.sample.datafetchers.CustomDataFetcherFactoryProvider
 import com.expedia.graphql.sample.datafetchers.SpringDataFetcherFactory
-import com.expedia.graphql.sample.directives.DirectiveWiringFactory
-import com.expedia.graphql.sample.directives.LowercaseDirectiveWiring
+import com.expedia.graphql.sample.directives.CustomDirectiveWiringFactory
 import com.expedia.graphql.sample.exceptions.CustomDataFetcherExceptionHandler
 import com.expedia.graphql.sample.extension.CustomSchemaGeneratorHooks
 import com.expedia.graphql.sample.mutation.Mutation
@@ -35,11 +34,11 @@ class Application {
     private val logger = LoggerFactory.getLogger(Application::class.java)
 
     @Bean
-    fun wiringFactory() = DirectiveWiringFactory()
+    fun wiringFactory() = CustomDirectiveWiringFactory()
 
     @Bean
-    fun hooks(validator: Validator, wiringFactory: DirectiveWiringFactory) =
-        CustomSchemaGeneratorHooks(validator, DirectiveWiringHelper(wiringFactory, mapOf("lowercase" to LowercaseDirectiveWiring())))
+    fun hooks(validator: Validator, wiringFactory: KotlinDirectiveWiringFactory) =
+        CustomSchemaGeneratorHooks(validator, wiringFactory)
 
     @Bean
     fun dataFetcherFactoryProvider(springDataFetcherFactory: SpringDataFetcherFactory, hooks: SchemaGeneratorHooks) =
@@ -59,6 +58,7 @@ class Application {
             .includeExtendedScalarTypes(true)
             .includeIntrospectionTypes(true)
             .includeSchemaDefintion(true)
+            .includeDirectives(true)
     )
 
     @Bean

example/src/main/kotlin/com/expedia/graphql/sample/directives/CakeOnlyDirectiveWiring.kt

@@ -0,0 +1,25 @@
+package com.expedia.graphql.sample.directives
+
+import com.expedia.graphql.directives.KotlinFieldDirectiveEnvironment
+import com.expedia.graphql.directives.KotlinSchemaDirectiveWiring
+import graphql.schema.DataFetcher
+import graphql.schema.GraphQLFieldDefinition
+
+class CakeOnlySchemaDirectiveWiring : KotlinSchemaDirectiveWiring {
+
+    @Throws(RuntimeException::class)
+    override fun onField(environment: KotlinFieldDirectiveEnvironment): GraphQLFieldDefinition {
+        val field = environment.element
+        val originalDataFetcher: DataFetcher<Any> = environment.getDataFetcher()
+
+        val cakeOnlyFetcher = DataFetcher<Any> { dataEnv ->
+            val strArg: String? = dataEnv.getArgument(environment.element.arguments[0].name) as String?
+            if (!"cake".equals(other = strArg, ignoreCase = true)) {
+                throw RuntimeException("The cake is a lie!")
+            }
+            originalDataFetcher.get(dataEnv)
+        }
+        environment.setDataFetcher(cakeOnlyFetcher)
+        return field
+    }
+}

example/src/main/kotlin/com/expedia/graphql/sample/directives/CakeOnlySchemaDirectiveWiring.kt

@@ -0,0 +1,23 @@
+package com.expedia.graphql.sample.directives
+
+import com.expedia.graphql.directives.KotlinDirectiveWiringFactory
+import com.expedia.graphql.directives.KotlinSchemaDirectiveEnvironment
+import com.expedia.graphql.directives.KotlinSchemaDirectiveWiring
+import com.google.common.base.CaseFormat
+import graphql.schema.GraphQLDirectiveContainer
+import kotlin.reflect.KClass
+
+class CustomDirectiveWiringFactory : KotlinDirectiveWiringFactory(manualWiring = mapOf<String, KotlinSchemaDirectiveWiring>("lowercase" to LowercaseSchemaDirectiveWiring())) {
+
+    private val stringEvalDirectiveWiring = StringEvalSchemaDirectiveWiring()
+    private val caleOnlyDirectiveWiring = CakeOnlySchemaDirectiveWiring()
+
+    override fun getSchemaDirectiveWiring(environment: KotlinSchemaDirectiveEnvironment<GraphQLDirectiveContainer>): KotlinSchemaDirectiveWiring? = when {
+        environment.directive.name == getDirectiveName(StringEval::class) -> stringEvalDirectiveWiring
+        environment.directive.name == getDirectiveName(CakeOnly::class) -> caleOnlyDirectiveWiring
+        else -> null
+    }
+}
+
+internal fun getDirectiveName(kClass: KClass<out Annotation>): String =
+    CaseFormat.UPPER_CAMEL.to(CaseFormat.LOWER_CAMEL, kClass.simpleName!!)

example/src/main/kotlin/com/expedia/graphql/sample/directives/CustomDirective.kt

@@ -0,0 +1,24 @@
+package com.expedia.graphql.sample.directives
+
+import com.expedia.graphql.directives.KotlinFieldDirectiveEnvironment
+import com.expedia.graphql.directives.KotlinSchemaDirectiveWiring
+import graphql.schema.DataFetcher
+import graphql.schema.DataFetcherFactories
+import graphql.schema.DataFetchingEnvironment
+import graphql.schema.GraphQLFieldDefinition
+import java.util.function.BiFunction
+
+class LowercaseSchemaDirectiveWiring : KotlinSchemaDirectiveWiring {
+
+    override fun onField(environment: KotlinFieldDirectiveEnvironment): GraphQLFieldDefinition {
+        val field = environment.element
+        val originalDataFetcher: DataFetcher<Any> = environment.getDataFetcher()
+
+        val lowerCaseFetcher = DataFetcherFactories.wrapDataFetcher(
+            originalDataFetcher,
+            BiFunction<DataFetchingEnvironment, Any, Any>{ _, value -> value.toString().toLowerCase() }
+        )
+        environment.setDataFetcher(lowerCaseFetcher)
+        return field
+    }
+}

example/src/main/kotlin/com/expedia/graphql/sample/directives/CustomDirectiveWiringFactory.kt

@@ -0,0 +1,50 @@
+package com.expedia.graphql.sample.directives
+
+import com.expedia.graphql.directives.KotlinFieldDirectiveEnvironment
+import com.expedia.graphql.directives.KotlinSchemaDirectiveEnvironment
+import com.expedia.graphql.directives.KotlinSchemaDirectiveWiring
+import graphql.schema.DataFetcher
+import graphql.schema.DataFetchingEnvironmentImpl.newDataFetchingEnvironment
+import graphql.schema.GraphQLArgument
+import graphql.schema.GraphQLFieldDefinition
+
+class StringEvalSchemaDirectiveWiring : KotlinSchemaDirectiveWiring {
+    private val directiveName = getDirectiveName(StringEval::class)
+
+    override fun onField(environment: KotlinFieldDirectiveEnvironment): GraphQLFieldDefinition {
+        val field = environment.element
+        val originalDataFetcher: DataFetcher<Any> = environment.getDataFetcher()
+
+        val defaultValueFetcher = DataFetcher<Any> { dataEnv ->
+            val newArguments = HashMap(dataEnv.arguments)
+            environment.element.arguments.associateWith {
+                dataEnv.getArgument(it.name) as String?
+            }.forEach { (graphQLArgumentType, value) ->
+                if (graphQLArgumentType.getDirective(directiveName).getArgument(StringEval::lowerCase.name).value as Boolean) {
+                    newArguments[graphQLArgumentType.name] = value?.toLowerCase()
+                }
+                if (value.isNullOrEmpty()) {
+                    newArguments[graphQLArgumentType.name] = graphQLArgumentType.defaultValue
+                }
+            }
+            val newEnv = newDataFetchingEnvironment(dataEnv)
+                    .arguments(newArguments)
+                    .build()
+            originalDataFetcher.get(newEnv)
+        }
+        environment.setDataFetcher(defaultValueFetcher)
+        return field
+    }
+
+    override fun onArgument(environment: KotlinSchemaDirectiveEnvironment<GraphQLArgument>): GraphQLArgument {
+        val argument = environment.element
+        val directive = environment.directive
+
+        val default = directive.getArgument(StringEval::default.name).value as String
+        return if (default.isNotEmpty()) {
+            argument.transform { it.defaultValue(default) }
+        } else {
+            argument
+        }
+    }
+}