Kotlin: Limit the KotlinSerializerCodec to Data, Value or Sealed Classes

JAVA-5035

Author: rozza

bson-kotlinx/src/main/kotlin/org/bson/codecs/kotlinx/KotlinSerializerCodec.kt

@@ -85,10 +85,14 @@ private constructor(
             serializersModule: SerializersModule = defaultSerializersModule,
             bsonConfiguration: BsonConfiguration = BsonConfiguration()
         ): Codec<T>? {
-            return try {
-                create(kClass, kClass.serializer(), serializersModule, bsonConfiguration)
-            } catch (exception: SerializationException) {
+            return if (!(kClass.isData || kClass.isValue || kClass.isSealed)) {
                 null
+            } else {
+                try {
+                    create(kClass, kClass.serializer(), serializersModule, bsonConfiguration)
+                } catch (exception: SerializationException) {
+                    null
+                }
             }
         }
 

bson-kotlinx/src/test/kotlin/org/bson/codecs/kotlinx/KotlinSerializerCodecProviderTest.kt

@@ -50,6 +50,12 @@ class KotlinSerializerCodecProviderTest {
         assertNull(codec)
     }
 
+    @Test
+    fun shouldReturnNullForSerializableButNotValueClassOrSealedOrDataClassTypes() {
+        assertNull(KotlinSerializerCodecProvider().get(DoubleArray::class.java, Bson.DEFAULT_CODEC_REGISTRY))
+        assertNull(KotlinSerializerCodecProvider().get(CharSequence::class.java, Bson.DEFAULT_CODEC_REGISTRY))
+    }
+
     @Test
     fun shouldReturnKotlinSerializerCodecUsingDefaultRegistry() {
         val codec = MongoClientSettings.getDefaultCodecRegistry().get(DataClassWithSimpleValues::class.java)