Fixed issue where reminders were not being updated correctly when upd…

…ating an event (#261)

# Background
There was a typo in the UpdateEventRequest which supplied `override`
instead of `overrides` for the reminders JSON object. However, this data
class was also necessarily duplicated. So I got rid of it and used the
pre-existing data class.

# License
<!-- Your PR comment must contain the following line for us to merge the
PR. -->
I confirm that this contribution is made under the terms of the MIT
license and that I have the authority necessary to make this
contribution on behalf of its copyright owner.

CHANGELOG.md

@@ -3,6 +3,7 @@
 ### Unreleased
 * Made `NylasClient` and its methods open to enable mocking in tests
 * 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,76 @@ 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(),
+      )
+
+      assertEquals("v3/grants/$grantId/events/$eventId", pathCaptor.firstValue)
+      assertEquals(Types.newParameterizedType(Response::class.java, Event::class.java), typeCaptor.firstValue)
+
+      // Parse both expected and actual JSON into Maps to compare structure while preserving field names
+      val jsonAdapter = JsonHelper.moshi().adapter(Map::class.java)
+      val actualJson = jsonAdapter.fromJson(requestBodyCaptor.firstValue)!!
+
+      val expectedJson = mapOf(
+        "reminders" to mapOf(
+          "use_default" to false,
+          "overrides" to listOf(
+            mapOf(
+              "reminder_minutes" to 15.0,
+              "reminder_method" to "email",
+            ),
+            mapOf(
+              "reminder_minutes" to 30.0,
+              "reminder_method" to "popup",
+            ),
+          ),
+        ),
+      )
+
+      assertEquals(expectedJson, actualJson)
+
+      // Also verify that the request can be correctly deserialized
+      val adapter = JsonHelper.moshi().adapter(UpdateEventRequest::class.java)
+      val actualRequest = adapter.fromJson(requestBodyCaptor.firstValue)
+      assertEquals(updateEventRequest, actualRequest)
+    }
+
     @Test
     fun `destroying a event calls requests with the correct params`() {
       val eventId = "event-123"