Merge pull request #682 from Netflix/feature/alias-fix

Kotlin2: Move aliases to the last position so they don't conflict with unnamed args

Author: mbossenbroek

graphql-dgs-codegen-core/src/integTest/kotlin/com/netflix/graphql/dgs/codegen/cases/constantsForInputTypes/expected/client/QueryProjection.kt

@@ -9,8 +9,8 @@ public class QueryProjection(
   inputValueSerializer: InputValueSerializerInterface? = null,
 ) : GraphQLProjection(inputValueSerializer) {
   public fun people(
-    _alias: String? = null,
     filter: PersonFilter? = default<QueryProjection, PersonFilter?>("filter"),
+    _alias: String? = null,
     _projection: PersonProjection.() -> PersonProjection,
   ): QueryProjection {
     field(_alias, "people", PersonProjection(inputValueSerializer), _projection, "filter" to filter)

graphql-dgs-codegen-core/src/integTest/kotlin/com/netflix/graphql/dgs/codegen/cases/constantsWithExtendedInputTypes/expected/client/QueryProjection.kt

@@ -9,8 +9,8 @@ public class QueryProjection(
   inputValueSerializer: InputValueSerializerInterface? = null,
 ) : GraphQLProjection(inputValueSerializer) {
   public fun people(
-    _alias: String? = null,
     filter: PersonFilter? = default<QueryProjection, PersonFilter?>("filter"),
+    _alias: String? = null,
     _projection: PersonProjection.() -> PersonProjection,
   ): QueryProjection {
     field(_alias, "people", PersonProjection(inputValueSerializer), _projection, "filter" to filter)

graphql-dgs-codegen-core/src/integTest/kotlin/com/netflix/graphql/dgs/codegen/cases/dataClassDocs/expected/client/QueryProjection.kt

@@ -9,8 +9,8 @@ public class QueryProjection(
   inputValueSerializer: InputValueSerializerInterface? = null,
 ) : GraphQLProjection(inputValueSerializer) {
   public fun search(
-    _alias: String? = null,
     movieFilter: MovieFilter,
+    _alias: String? = null,
     _projection: MovieProjection.() -> MovieProjection,
   ): QueryProjection {
     field(_alias, "search", MovieProjection(inputValueSerializer), _projection, "movieFilter" to

graphql-dgs-codegen-core/src/integTest/kotlin/com/netflix/graphql/dgs/codegen/cases/dataClassFieldDocs/expected/client/QueryProjection.kt

@@ -9,8 +9,8 @@ public class QueryProjection(
   inputValueSerializer: InputValueSerializerInterface? = null,
 ) : GraphQLProjection(inputValueSerializer) {
   public fun search(
-    _alias: String? = null,
     movieFilter: MovieFilter,
+    _alias: String? = null,
     _projection: MovieProjection.() -> MovieProjection,
   ): QueryProjection {
     field(_alias, "search", MovieProjection(inputValueSerializer), _projection, "movieFilter" to

graphql-dgs-codegen-core/src/integTest/kotlin/com/netflix/graphql/dgs/codegen/cases/projectionWithTypeAndArgs/expected/client/QueryProjection.kt

@@ -9,10 +9,10 @@ public class QueryProjection(
   inputValueSerializer: InputValueSerializerInterface? = null,
 ) : GraphQLProjection(inputValueSerializer) {
   public fun person(
-    _alias: String? = null,
     a1: String? = default<QueryProjection, String?>("a1"),
     a2: String,
     a3: I? = default<QueryProjection, I?>("a3"),
+    _alias: String? = null,
     _projection: PersonProjection.() -> PersonProjection,
   ): QueryProjection {
     field(_alias, "person", PersonProjection(inputValueSerializer), _projection, "a1" to a1 , "a2"

graphql-dgs-codegen-core/src/integTest/kotlin/com/netflix/graphql/dgs/codegen/cases/projectionWithTypeAndArgs/test/QueryTest.kt

@@ -53,6 +53,28 @@ class QueryTest {
         )
     }
 
+    @Test
+    fun testQueryWithUnnamedArgs() {
+        val query = DgsClient.buildQuery {
+            person("a1", "a2") {
+                firstname
+            }
+        }
+
+        assertEquals(
+            """{
+            |  __typename
+            |  person(a1: "a1", a2: "a2") {
+            |    __typename
+            |    firstname
+            |  }
+            |}
+            |
+            """.trimMargin(),
+            query
+        )
+    }
+
     @Test
     fun testQueryWithAlias() {
         val query = DgsClient.buildQuery {

graphql-dgs-codegen-core/src/integTest/kotlin/com/netflix/graphql/dgs/codegen/cases/unionTypesWithoutInterfaceCanDeserialize/expected/client/QueryProjection.kt

@@ -8,8 +8,8 @@ public class QueryProjection(
   inputValueSerializer: InputValueSerializerInterface? = null,
 ) : GraphQLProjection(inputValueSerializer) {
   public fun search(
-    _alias: String? = null,
     text: String,
+    _alias: String? = null,
     _projection: SearchResultPageProjection.() -> SearchResultPageProjection,
   ): QueryProjection {
     field(_alias, "search", SearchResultPageProjection(inputValueSerializer), _projection, "text" to

graphql-dgs-codegen-core/src/main/kotlin/com/netflix/graphql/dgs/codegen/generators/kotlin2/GenerateKotlin2ClientTypes.kt

@@ -121,8 +121,8 @@ fun generateKotlin2ClientTypes(
                             val (projectionType, projection) = projectionType(config.packageNameClient, projectionTypeName)
 
                             FunSpec.builder(field.name)
-                                .addParameter(ParameterSpec.builder("_alias", String::class.asTypeName().copy(nullable = true)).defaultValue("null").build())
                                 .addInputArgs(config, typeLookup, typeName, field.inputValueDefinitions)
+                                .addParameter(ParameterSpec.builder("_alias", String::class.asTypeName().copy(nullable = true)).defaultValue("null").build())
                                 .addParameter(projection)
                                 .returns(typeName)
                                 .addStatement(