fix(amazonq): fix warnings when metrics fields are null

Author: rli

plugins/amazonq/shared/jetbrains-community/src/software/aws/toolkits/jetbrains/services/amazonq/lsp/AmazonQLanguageClientImpl.kt

@@ -77,7 +77,7 @@ class AmazonQLanguageClientImpl(private val project: Project) : AmazonQLanguageC
             val name = telemetryMap["name"] as? String ?: return
 
             @Suppress("UNCHECKED_CAST")
-            val data = telemetryMap["data"] as? Map<String, Any> ?: return
+            val data = telemetryMap["data"] as? Map<String, Any?> ?: return
 
             TelemetryService.getInstance().record(project) {
                 datum(name) {
@@ -90,7 +90,7 @@ class AmazonQLanguageClientImpl(private val project: Project) : AmazonQLanguageC
                     }
 
                     data.forEach { (key, value) ->
-                        metadata(key, value.toString())
+                        metadata(key, value?.toString() ?: "null")
                     }
                 }
             }

plugins/amazonq/shared/jetbrains-community/tst/software/aws/toolkits/jetbrains/services/amazonq/lsp/AmazonQLanguageClientImplTest.kt

@@ -81,6 +81,36 @@ class AmazonQLanguageClientImplTest {
         )
     }
 
+    @Test
+    fun `telemetryEvent handles null`() {
+        val telemetryService = mockk<TelemetryService>(relaxed = true)
+        mockkObject(TelemetryService)
+        every { TelemetryService.getInstance() } returns telemetryService
+
+        val builderCaptor = slot<MetricEvent.Builder.() -> Unit>()
+        every { telemetryService.record(project, capture(builderCaptor)) } returns Unit
+
+        val event = mapOf(
+            "name" to "test_event",
+            "data" to mapOf(
+                "key1" to null,
+            )
+        )
+
+        sut.telemetryEvent(event)
+
+        val builder = DefaultMetricEvent.builder()
+        builderCaptor.captured.invoke(builder)
+
+        val metricEvent = builder.build()
+        val datum = metricEvent.data.first()
+
+        assertThat(datum.name).isEqualTo("test_event")
+        assertThat(datum.metadata).contains(
+            entry("key1", "null"),
+        )
+    }
+
     @Test
     fun `telemetryEvent handles event with result field`() {
         val telemetryService = mockk<TelemetryService>(relaxed = true)