added missing LocalTime converters and test

Author: Jolanrensen

core/src/main/kotlin/org/jetbrains/kotlinx/dataframe/impl/api/convert.kt

@@ -15,6 +15,7 @@ import kotlinx.datetime.toJavaLocalTime
 import kotlinx.datetime.toKotlinInstant
 import kotlinx.datetime.toKotlinLocalDate
 import kotlinx.datetime.toKotlinLocalDateTime
+import kotlinx.datetime.toKotlinLocalTime
 import kotlinx.datetime.toLocalDateTime
 import org.jetbrains.kotlinx.dataframe.AnyCol
 import org.jetbrains.kotlinx.dataframe.DataColumn
@@ -350,6 +351,8 @@ internal fun createConverter(from: KType, to: KType, options: ParserOptions? = n
 
                 LocalDate::class -> convert<Long> { it.toLocalDate(defaultTimeZone) }
 
+                LocalTime::class -> convert<Long> { it.toLocalTime(defaultTimeZone) }
+
                 Instant::class -> convert<Long> { Instant.fromEpochMilliseconds(it) }
 
                 JavaLocalDateTime::class -> convert<Long> {
@@ -504,6 +507,16 @@ internal fun createConverter(from: KType, to: KType, options: ParserOptions? = n
                 else -> null
             }
 
+            LocalTime::class -> when (toClass) {
+                JavaLocalTime::class -> convert<LocalTime> { it.toJavaLocalTime() }
+                else -> null
+            }
+
+            JavaLocalTime::class -> when (toClass) {
+                LocalTime::class -> convert<JavaLocalTime> { it.toKotlinLocalTime() }
+                else -> null
+            }
+
             URL::class -> when (toClass) {
                 IMG::class -> convert<URL> { IMG(it.toString()) }
                 IFRAME::class -> convert<URL> { IFRAME(it.toString()) }

core/src/test/kotlin/org/jetbrains/kotlinx/dataframe/api/convert.kt

@@ -16,9 +16,20 @@ import org.jetbrains.kotlinx.dataframe.hasNulls
 import org.junit.Test
 import kotlin.reflect.typeOf
 import kotlin.time.Duration.Companion.hours
+import java.time.LocalTime as JavaLocalTime
 
 class ConvertTests {
 
+    @Test
+    fun `convert LocalTime Kotlin to Java and back`() {
+        val time by columnOf(LocalTime(11, 22, 33))
+        val converted = time.toDataFrame().convert { time }.to<JavaLocalTime>()
+        converted[time][0] shouldBe JavaLocalTime.of(11, 22, 33)
+
+        val convertedBack = converted.convert(time).to<LocalTime>()
+        convertedBack[time][0] shouldBe time[0]
+    }
+
     @Test
     fun `convert nullable strings to time`() {
         val time by columnOf("11?22?33", null)