[Kotlin client] Improve JSON parser (#2981)

* [Kotlin client] Improve JSON parser

* fix import in model for threetenbp date

samples/client/petstore/kotlin-string/src/main/kotlin/org/openapitools/client/infrastructure/ApiClient.kt

@@ -1,9 +1,8 @@
 package org.openapitools.client.infrastructure
 
-import com.squareup.moshi.FromJson
 import com.squareup.moshi.KotlinJsonAdapterFactory
 import com.squareup.moshi.Moshi
-import com.squareup.moshi.ToJson
+import com.squareup.moshi.Rfc3339DateJsonAdapter
 import okhttp3.OkHttpClient
 import okhttp3.RequestBody
 import okhttp3.MediaType
@@ -12,7 +11,7 @@ import okhttp3.HttpUrl
 import okhttp3.ResponseBody
 import okhttp3.Request
 import java.io.File
-import java.util.UUID
+import java.util.Date
 
 open class ApiClient(val baseUrl: String) {
     companion object {
@@ -63,12 +62,11 @@ open class ApiClient(val baseUrl: String) {
             return null
         }
         return when(mediaType) {
-            JsonMediaType -> Moshi.Builder().add(object {
-                    @ToJson
-                    fun toJson(uuid: UUID) = uuid.toString()
-                    @FromJson
-                    fun fromJson(s: String) = UUID.fromString(s)
-                })
+            JsonMediaType -> Moshi.Builder()
+                .add(Date::class.java, Rfc3339DateJsonAdapter().nullSafe())
+                .add(LocalDateTimeAdapter())
+                .add(LocalDateAdapter())
+                .add(UUIDAdapter())
                 .add(ByteArrayAdapter())
                 .add(KotlinJsonAdapterFactory())
                 .build().adapter(T::class.java).fromJson(bodyContent)

samples/client/petstore/kotlin-string/src/main/kotlin/org/openapitools/client/infrastructure/LocalDateAdapter.kt

@@ -0,0 +1,19 @@
+package org.openapitools.client.infrastructure
+
+import com.squareup.moshi.FromJson
+import com.squareup.moshi.ToJson
+import java.time.LocalDate
+import java.time.format.DateTimeFormatter
+
+class LocalDateAdapter {
+    @ToJson
+    fun toJson(value: LocalDate): String {
+        return DateTimeFormatter.ISO_LOCAL_DATE.format(value)
+    }
+
+    @FromJson
+    fun fromJson(value: String): LocalDate {
+        return LocalDate.parse(value, DateTimeFormatter.ISO_LOCAL_DATE)
+    }
+
+}

samples/client/petstore/kotlin-string/src/main/kotlin/org/openapitools/client/infrastructure/LocalDateTimeAdapter.kt

@@ -0,0 +1,19 @@
+package org.openapitools.client.infrastructure
+
+import com.squareup.moshi.FromJson
+import com.squareup.moshi.ToJson
+import java.time.LocalDateTime
+import java.time.format.DateTimeFormatter
+
+class LocalDateTimeAdapter {
+    @ToJson
+    fun toJson(value: LocalDateTime): String {
+        return DateTimeFormatter.ISO_LOCAL_DATE_TIME.format(value)
+    }
+
+    @FromJson
+    fun fromJson(value: String): LocalDateTime {
+        return LocalDateTime.parse(value, DateTimeFormatter.ISO_LOCAL_DATE_TIME)
+    }
+
+}

samples/client/petstore/kotlin-string/src/main/kotlin/org/openapitools/client/infrastructure/Serializer.kt

@@ -3,12 +3,16 @@ package org.openapitools.client.infrastructure
 import com.squareup.moshi.KotlinJsonAdapterFactory
 import com.squareup.moshi.Moshi
 import com.squareup.moshi.Rfc3339DateJsonAdapter
-import java.util.*
+import java.util.Date
 
 object Serializer {
     @JvmStatic
     val moshi: Moshi = Moshi.Builder()
-        .add(KotlinJsonAdapterFactory())
         .add(Date::class.java, Rfc3339DateJsonAdapter().nullSafe())
+        .add(LocalDateTimeAdapter())
+        .add(LocalDateAdapter())
+        .add(UUIDAdapter())
+        .add(ByteArrayAdapter())
+        .add(KotlinJsonAdapterFactory())
         .build()
 }

samples/client/petstore/kotlin-string/src/main/kotlin/org/openapitools/client/infrastructure/UUIDAdapter.kt

@@ -0,0 +1,13 @@
+package org.openapitools.client.infrastructure
+
+import com.squareup.moshi.FromJson
+import com.squareup.moshi.ToJson
+import java.util.UUID
+
+class UUIDAdapter {
+    @ToJson
+    fun toJson(uuid: UUID) = uuid.toString()
+
+    @FromJson
+    fun fromJson(s: String) = UUID.fromString(s)
+}

samples/client/petstore/kotlin-string/src/test/kotlin/org/openapitools/client/PetApiTest.kt

@@ -2,8 +2,11 @@ package org.openapitools.client
 
 import io.kotlintest.shouldBe
 import io.kotlintest.matchers.numerics.shouldBeGreaterThan
+import io.kotlintest.matchers.string.shouldContain
+import io.kotlintest.shouldThrow
 import io.kotlintest.specs.ShouldSpec
 import org.openapitools.client.apis.PetApi
+import org.openapitools.client.infrastructure.ClientException
 import org.openapitools.client.models.Category
 import org.openapitools.client.models.Pet
 import org.openapitools.client.models.Tag
@@ -11,8 +14,11 @@ import org.openapitools.client.models.Tag
 class PetApiTest : ShouldSpec() {
     init {
 
+        val petId:Long = 10006
+        val api = PetApi()
+
         should("add a pet") {
-            val petId:Long = 10006
+
             val pet = Pet(
                     id = petId,
                     name = "kotlin client test",
@@ -20,15 +26,11 @@ class PetApiTest : ShouldSpec() {
                     category = Category(petId, "test kotlin category"),
                     tags = arrayOf(Tag(petId, "test kotlin tag"))
             )
-
-            val api = PetApi()
             api.addPet(pet)
 
         }
 
         should("get pet by id") {
-            val petId: Long = 10006
-            val api = PetApi()
             val result = api.getPetById(petId)
 
             result.id shouldBe (petId)
@@ -42,7 +44,6 @@ class PetApiTest : ShouldSpec() {
         }
 
         should("find pet by status") {
-            val api = PetApi()
             val result = api.findPetsByStatus(arrayOf("available"))
 
             result.size.shouldBeGreaterThan(0)
@@ -58,15 +59,12 @@ class PetApiTest : ShouldSpec() {
         }
 
         should("update a pet") {
-            val petId:Long = 10007
             val pet = Pet(
                     id = petId,
                     name = "kotlin client updatePet",
                     status = Pet.Status.pending,
                     photoUrls = arrayOf("http://test_kotlin_unit_test.com")
             )
-
-            val api = PetApi()
             api.updatePet(pet)
 
             // verify updated Pet
@@ -77,14 +75,10 @@ class PetApiTest : ShouldSpec() {
 
         }
 
-        //TODO the test fail cause client doesn't support other JSON contentType/Accept
-        /*
         should("update a pet with form") {
-            val petId:Long = 10007
             val name = "kotlin client updatePet with Form"
             val status = "pending"
 
-            val api = PetApi()
             api.updatePetWithForm(petId, name, status)
 
             // verify updated Pet
@@ -94,7 +88,16 @@ class PetApiTest : ShouldSpec() {
             result.status shouldBe (Pet.Status.pending)
 
         }
-        */
+
+        should("delete a pet") {
+            api.deletePet(petId, "apiKey")
+
+            // verify updated Pet
+            val exception = shouldThrow<ClientException> {
+                api.getPetById(petId)
+            }
+            exception.message?.shouldContain("Pet not found")
+        }
 
     }