Fixed issue where reminders were not being updated correctly when updating an event

Author: AaronDDM

CHANGELOG.md

@@ -2,6 +2,7 @@
 
 ### Unreleased
 * Added pagination support for folders
+* Fixed issue where reminders were not being updated correctly when updating an event
 
 ### [2.5.2] - Released 2024-12-02
 * Added support for `skypeForConsumer` as conferencing provider

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

@@ -495,23 +495,6 @@ data class UpdateEventRequest(
     }
   }
 
-  /**
-   * Class representing the reminders field of an event.
-   */
-  data class Reminders(
-    /**
-     * Whether to use the default reminders for the calendar.
-     * When true, uses the default reminder settings for the calendar
-     */
-    @Json(name = "use_default")
-    val useDefault: Boolean? = null,
-    /**
-     * A list of reminders for the event if useDefault is set to false.
-     */
-    @Json(name = "override")
-    val override: List<ReminderOverride>? = null,
-  )
-
   /**
    * Builder for [UpdateEventRequest].
    */

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

@@ -345,6 +345,68 @@ class EventsTests {
       assertEquals(adapter.toJson(updateEventRequest), requestBodyCaptor.firstValue)
     }
 
+    @Test
+    fun `updating event reminders calls requests with the correct params`() {
+      val eventId = "event-123"
+      val updateEventRequest =
+        UpdateEventRequest(
+          reminders = Reminders(
+            useDefault = false,
+            overrides = listOf(
+              ReminderOverride(
+                reminderMinutes = 15,
+                reminderMethod = ReminderMethod.EMAIL,
+              ),
+              ReminderOverride(
+                reminderMinutes = 30,
+                reminderMethod = ReminderMethod.POPUP,
+              ),
+            ),
+          ),
+        )
+      val updateEventQueryParams =
+        UpdateEventQueryParams(
+          calendarId = "calendar-id",
+          notifyParticipants = true,
+        )
+
+      events.update(grantId, eventId, updateEventRequest, updateEventQueryParams)
+      val pathCaptor = argumentCaptor<String>()
+      val typeCaptor = argumentCaptor<Type>()
+      val requestBodyCaptor = argumentCaptor<String>()
+      val queryParamCaptor = argumentCaptor<UpdateEventQueryParams>()
+      val overrideParamCaptor = argumentCaptor<RequestOverrides>()
+      verify(mockNylasClient).executePut<ListResponse<Event>>(
+        pathCaptor.capture(),
+        typeCaptor.capture(),
+        requestBodyCaptor.capture(),
+        queryParamCaptor.capture(),
+        overrideParamCaptor.capture(),
+      )
+
+      val expectedJson = """
+        {
+          "reminders": {
+            "use_default": false,
+            "overrides": [
+              {
+                "reminder_minutes": 15,
+                "reminder_method": "email"
+              },
+              {
+                "reminder_minutes": 30,
+                "reminder_method": "popup"
+              }
+            ]
+          }
+        }
+      """.trimIndent()
+
+      assertEquals("v3/grants/$grantId/events/$eventId", pathCaptor.firstValue)
+      assertEquals(Types.newParameterizedType(Response::class.java, Event::class.java), typeCaptor.firstValue)
+      assertEquals(expectedJson, requestBodyCaptor.firstValue)
+    }
+
     @Test
     fun `destroying a event calls requests with the correct params`() {
       val eventId = "event-123"