Merge pull request #429 from dbinit/dbinit/fix-event-skeleton-ts

Make ts optional in SlackEventSkeleton

Author: jaredstehler

slack-base/src/main/java/com/hubspot/slack/client/models/events/SlackEventSkeletonIF.java

@@ -15,4 +15,10 @@
 public interface SlackEventSkeletonIF extends SlackEvent {
   @JsonProperty("channel")
   Optional<String> getChannelMaybe();
+
+  // Many event types deserialized as skeletons do not have a ts, so we manually set it as null.
+  @Override
+  default String getTs() {
+    return null;
+  }
 }

slack-base/src/test/java/com/hubspot/slack/client/models/events/json/EventDeserializerTest.java

@@ -275,6 +275,13 @@ public void itCanDeserLinkSharedEvent() throws IOException {
       .isEqualTo(event);
   }
 
+  @Test
+  public void itCanDeserReactionAddedEvent() throws IOException {
+    SlackEvent event = fetchAndDeserializeSlackEvent("reaction_added.json");
+    assertThat(event.getType()).isEqualTo(SlackEventType.REACTION_ADDED);
+    assertThat(event.getTs()).isNull();
+  }
+
   @Test
   public void itCanDeserBotMessageEventCallbacks() throws IOException {
     SlackEvent event = fetchAndDeserializeSlackEvent("bot_message_event_callback.json");

slack-base/src/test/resources/reaction_added.json

@@ -0,0 +1,23 @@
+{
+  "token": "redacted",
+  "team_id": "T024G0P55",
+  "api_app_id": "ALZH69H5Z",
+  "event": {
+    "type": "reaction_added",
+    "user": "U0374K33CR2",
+    "reaction": "+1::skin-tone-4",
+    "item": {
+      "type": "message",
+      "channel": "C0AFGUN15AL",
+      "ts": "1771357246.016889"
+    },
+    "item_user": "U03TL2T7VNC",
+    "event_ts": "1771358418.002600"
+  },
+  "type": "event_callback",
+  "event_id": "Ev0AFC3DRA3V",
+  "event_time": 1771358418,
+  "authed_users": [
+    "UMB1NLKNC"
+  ]
+}