Set TENTATIVE as deprecated, use MAYBE instead

Author: googsvg

src/main/kotlin/com/nylas/models/EventStatus.kt

@@ -12,6 +12,10 @@ enum class EventStatus {
   @Json(name = "maybe")
   MAYBE,
 
+  @Deprecated(
+    message = "Use MAYBE instead. TENTATIVE is a legacy alias and will be removed in a future release.",
+    replaceWith = ReplaceWith("MAYBE"),
+  )
   @Json(name = "tentative")
   TENTATIVE,
 

src/main/kotlin/com/nylas/util/EventStatusJsonAdapter.kt

@@ -5,19 +5,33 @@ import com.squareup.moshi.FromJson
 import com.squareup.moshi.JsonReader
 import com.squareup.moshi.JsonWriter
 import com.squareup.moshi.ToJson
-import com.squareup.moshi.adapters.EnumJsonAdapter
 
 /**
- * Handles EventStatus values without failing deserialization when the API returns a new status.
+ * Normalizes legacy event status values and avoids failing deserialization on new values.
  */
 class EventStatusJsonAdapter {
-  private val delegate = EnumJsonAdapter.create(EventStatus::class.java).withUnknownFallback(null)
-
   @FromJson
-  fun fromJson(reader: JsonReader): EventStatus? = delegate.fromJson(reader)
+  fun fromJson(reader: JsonReader): EventStatus? {
+    if (reader.peek() == JsonReader.Token.NULL) {
+      return reader.nextNull()
+    }
+
+    return when (reader.nextString()) {
+      "confirmed" -> EventStatus.CONFIRMED
+      "maybe", "tentative" -> EventStatus.MAYBE
+      "cancelled" -> EventStatus.CANCELLED
+      else -> null
+    }
+  }
 
   @ToJson
+  @Suppress("DEPRECATION")
   fun toJson(writer: JsonWriter, value: EventStatus?) {
-    delegate.toJson(writer, value)
+    when (value) {
+      null -> writer.nullValue()
+      EventStatus.CONFIRMED -> writer.value("confirmed")
+      EventStatus.MAYBE, EventStatus.TENTATIVE -> writer.value("maybe")
+      EventStatus.CANCELLED -> writer.value("cancelled")
+    }
   }
 }

src/test/kotlin/com/nylas/resources/EventsTests.kt

@@ -248,6 +248,31 @@ class EventsTests {
       assertEquals(EventStatus.MAYBE, event.status)
     }
 
+    @Test
+    fun `Event deserializes tentative status as maybe`() {
+      val adapter = JsonHelper.moshi().adapter(Event::class.java)
+      val jsonBuffer =
+        Buffer().writeUtf8(
+          """
+          {
+            "id": "5d3qmne77v32r8l4phyuksl2x",
+            "grant_id": "41009df5-bf11-4c97-aa18-b285b5f2e386",
+            "calendar_id": "7d93zl2palhxqdy6e5qinsakt",
+            "object": "event",
+            "status": "tentative",
+            "when": {
+              "date": "2024-06-18",
+              "object": "date"
+            }
+          }
+          """.trimIndent(),
+        )
+
+      val event = adapter.fromJson(jsonBuffer)!!
+
+      assertEquals(EventStatus.MAYBE, event.status)
+    }
+
     @Test
     fun `Event deserializes unknown status as null`() {
       val adapter = JsonHelper.moshi().adapter(Event::class.java)
@@ -273,6 +298,15 @@ class EventsTests {
       assertEquals(null, event.status)
     }
 
+    @Suppress("DEPRECATION")
+    @Test
+    fun `Event status serializes tentative as maybe`() {
+      val adapter = JsonHelper.moshi().adapter(EventStatus::class.java)
+
+      assertEquals("\"maybe\"", adapter.toJson(EventStatus.MAYBE))
+      assertEquals("\"maybe\"", adapter.toJson(EventStatus.TENTATIVE))
+    }
+
     @Test
     fun `CreateEventAutoConferencingProvider serializes properly`() {
       val adapter = JsonHelper.moshi().adapter(CreateEventAutoConferencingProvider::class.java)