openapi-processor/openapi-processor-spring#150, parse multi level generics of response

Author: hauner

openapi-processor-core/src/main/kotlin/io/openapiprocessor/core/processor/mapping/v2/MappingConverter.kt

@@ -170,6 +170,20 @@ class MappingConverter(val mapping: MappingV2) {
         return AddParameterTypeMapping(mapping.sourceType!!, typeMapping, annotation)
     }
 
+    private fun convertResponse(source: Response): Mapping {
+        val (mapping, genericTypes, genericTypeNames) = parseMapping(source.content, source.generics)
+
+        val typeMapping = TypeMapping(
+            null,
+            null,
+            resolvePackageVariable(mapping.targetType!!),
+            resolvePackageVariable(genericTypeNames),
+            genericTypes
+        )
+
+        return ResponseTypeMapping (mapping.sourceType!!, typeMapping)
+    }
+
     data class ParsedMapping(
         val mapping: ParserMapping,
         val genericTypes: List<TargetType>,
@@ -218,24 +232,6 @@ class MappingConverter(val mapping: MappingV2) {
             .collect(Collectors.toList())
     }
 
-    private fun convertResponse(source: Response): Mapping {
-        val mapping = parseMapping(source.content)
-
-        val targetGenericTypes = mapping.targetGenericTypes.toMutableList()
-        if (targetGenericTypes.isEmpty() && source.generics != null) {
-            targetGenericTypes.addAll(source.generics)
-        }
-
-        val typeMapping = TypeMapping(
-            null,
-            null,
-            resolvePackageVariable(mapping.targetType!!),
-            resolvePackageVariable(targetGenericTypes)
-        )
-
-        return ResponseTypeMapping (mapping.sourceType!!, typeMapping)
-    }
-
     private fun convertPath(path: String, source: Path): Mapping {
         val result = ArrayList<Mapping>()
 

openapi-processor-core/src/test/kotlin/io/openapiprocessor/core/processor/mapping/v2/MappingConverterGenericsSpec.kt

@@ -7,9 +7,11 @@ package io.openapiprocessor.core.processor.mapping.v2
 
 import io.kotest.core.spec.IsolationMode
 import io.kotest.core.spec.style.StringSpec
+import io.kotest.matchers.nulls.shouldBeNull
 import io.kotest.matchers.shouldBe
 import io.openapiprocessor.core.converter.mapping.AddParameterTypeMapping
 import io.openapiprocessor.core.converter.mapping.ParameterTypeMapping
+import io.openapiprocessor.core.converter.mapping.ResponseTypeMapping
 import io.openapiprocessor.core.converter.mapping.TypeMapping
 import io.openapiprocessor.core.processor.MappingConverter
 import io.openapiprocessor.core.processor.MappingReader
@@ -241,4 +243,38 @@ class MappingConverterGenericsSpec: StringSpec({
         coll.typeName shouldBe "java.util.Collection"
         coll.genericTypes[0].typeName shouldBe "generated.String"
     }
+
+    "read response with nested generic parameters & package ref" {
+        val yaml = """
+                   |openapi-processor-mapping: v2
+                   |
+                   |options:
+                   |  package-name: generated
+                   |
+                   |map:
+                   |  responses:
+                   |    - content: foo/bar => java.util.Map
+                   |      generics:
+                   |        - java.lang.String
+                   |        - java.util.Collection<{package-name}.String>
+                   """.trimMargin()
+
+        // when:
+        val mapping = reader.read (yaml)
+        val mappings = converter.convert (mapping)
+
+        // then:
+        val type = mappings.first() as ResponseTypeMapping
+        type.contentType shouldBe "foo/bar"
+        val tm = type.mapping
+        tm.sourceTypeName.shouldBeNull()
+        tm.sourceTypeFormat.shouldBeNull()
+        tm.targetTypeName shouldBe "java.util.Map"
+        tm.genericTypes.size shouldBe 2
+        tm.genericTypes[0].typeName shouldBe "java.lang.String"
+
+        val coll = tm.genericTypes[1]
+        coll.typeName shouldBe "java.util.Collection"
+        coll.genericTypes[0].typeName shouldBe "generated.String"
+    }
 })