Fixed handling of null and JsonValue as well as FasterXML#904

Author: k163377

src/main/kotlin/com/fasterxml/jackson/module/kotlin/KotlinSerializers.kt

@@ -58,7 +58,7 @@ object ValueClassUnboxSerializer : StdSerializer<Any>(Any::class.java) {
         val unboxed = value::class.java.getMethod("unbox-impl").invoke(value)
 
         if (unboxed == null) {
-            provider.findNullValueSerializer(null).serialize(null, gen, provider)
+            provider.defaultSerializeNull(gen)
             return
         }
 
@@ -77,8 +77,8 @@ internal sealed class ValueClassSerializer<T : Any>(t: Class<T>) : StdSerializer
             // As shown in the processing of the factory function, jsonValueGetter is always a static method.
             val jsonValue: Any? = staticJsonValueGetter.invoke(null, unboxed)
             jsonValue
-                ?.let { provider.findValueSerializer(it::class.java).serialize(it, gen, provider) }
-                ?: provider.findNullValueSerializer(null).serialize(null, gen, provider)
+                ?.let { provider.defaultSerializeValue(it, gen) }
+                ?: provider.defaultSerializeNull(gen)
         }
     }
 

src/test/kotlin/com/fasterxml/jackson/module/kotlin/test/github/GitHub873.kt

@@ -1,11 +1,22 @@
 package com.fasterxml.jackson.module.kotlin.test.github
 
+import com.fasterxml.jackson.annotation.JsonValue
 import com.fasterxml.jackson.module.kotlin.defaultMapper
 import com.fasterxml.jackson.module.kotlin.jacksonObjectMapper
 import com.fasterxml.jackson.module.kotlin.readValue
 import kotlin.test.Test
 
 class GitHub873 {
+    @JvmInline
+    value class Person(
+        val properties: Map<String, Any>,
+    )
+
+    data class TimestampedPerson(
+        val timestamp: Long,
+        val person: Person,
+    )
+
     @Test
     fun `should serialize value class`() {
 
@@ -35,12 +46,18 @@ class GitHub873 {
     }
 
     @JvmInline
-    value class Person(
-        val properties: Map<String, Any>,
-    )
+    value class MapAsJsonValue(val value: String) {
+        @get:JsonValue
+        val jsonValue get() = mapOf("key" to value)
+    }
 
-    data class TimestampedPerson(
-        val timestamp: Long,
-        val person: Person,
-    )
+    data class JsonValueWrapper(val value: MapAsJsonValue)
+
+    @Test
+    fun `JsonValue is serialized in the same way`() {
+        val data = JsonValueWrapper(MapAsJsonValue("value"))
+        val json = defaultMapper.writeValueAsString(data)
+
+        assert("""{"value":{"key":"value"}}""" == json)
+    }
 }