Add missing properties to ChannelDeletedEvent (#994)

Author: seratch

json-logs/samples/events/ChannelDeletedPayload.json

@@ -25,6 +25,8 @@
   "event_context": "",
   "event": {
     "type": "channel_deleted",
-    "channel": ""
+    "channel": "",
+    "actor_id": "",
+    "event_ts": ""
   }
 }

json-logs/samples/rtm/ChannelDeletedEvent.json

@@ -1,4 +1,6 @@
 {
   "type": "channel_deleted",
-  "channel": ""
+  "channel": "",
+  "actor_id": "",
+  "event_ts": ""
 }

slack-api-model/src/main/java/com/slack/api/model/event/ChannelDeletedEvent.java

@@ -15,4 +15,6 @@ public class ChannelDeletedEvent implements Event {
 
     private final String type = TYPE_NAME;
     private String channel;
+    private String actorId;
+    private String eventTs;
 }

slack-api-model/src/test/java/test_locally/api/model/event/ChannelCreatedEventTest.java

@@ -22,7 +22,12 @@ public void deserialize() {
                 "    \"type\": \"channel_created\",\n" +
                 "    \"channel\": {\n" +
                 "        \"id\": \"C024BE91L\",\n" +
+                "        \"is_channel\": true,\n" +
+                "        \"is_org_shared\": false,\n" +
+                "        \"is_shared\": false,\n" +
                 "        \"name\": \"fun\",\n" +
+                "        \"name\": \"fun\",\n" +
+                "        \"name_normalized\": \"fun\",\n" +
                 "        \"created\": 1360782804,\n" +
                 "        \"creator\": \"U024BE7LH\"\n" +
                 "    }\n" +

slack-api-model/src/test/java/test_locally/api/model/event/ChannelDeletedEventTest.java

@@ -18,8 +18,10 @@ public void typeName() {
     @Test
     public void deserialize() {
         String json = "{\n" +
-                "    \"type\": \"channel_deleted\",\n" +
-                "    \"channel\": \"C024BE91L\"\n" +
+                "  \"channel\": \"C024BE91L\",\n" +
+                "  \"actor_id\": \"U03E94MK0\",\n" +
+                "  \"type\": \"channel_deleted\",\n" +
+                "  \"event_ts\": \"1654751431.047500\"\n" +
                 "}";
         ChannelDeletedEvent event = GsonFactory.createSnakeCase().fromJson(json, ChannelDeletedEvent.class);
         assertThat(event.getType(), is("channel_deleted"));

slack-api-model/src/test/java/test_locally/api/model/event/ChannelRenameEventTest.java

@@ -19,18 +19,22 @@ public void typeName() {
     @Test
     public void deserialize() {
         String json = "{\n" +
-                "    \"type\": \"channel_rename\",\n" +
-                "    \"channel\": {\n" +
-                "        \"id\":\"C02ELGNBH\",\n" +
-                "        \"name\":\"new_name\",\n" +
-                "        \"created\":1360782804\n" +
-                "    }\n" +
+                "  \"type\": \"channel_rename\",\n" +
+                "  \"channel\": {\n" +
+                "    \"id\": \"C02ELGNBH\",\n" +
+                "    \"is_channel\": true,\n" +
+                "    \"is_mpim\": false,\n" +
+                "    \"name\": \"channel-renamed\",\n" +
+                "    \"name_normalized\": \"channel-renamed\",\n" +
+                "    \"created\": 1360782804\n" +
+                "  },\n" +
+                "  \"event_ts\": \"1654751422.047200\"\n" +
                 "}";
         ChannelRenameEvent event = GsonFactory.createSnakeCase().fromJson(json, ChannelRenameEvent.class);
         assertThat(event.getType(), is("channel_rename"));
         assertThat(event.getChannel(), is(notNullValue()));
         assertThat(event.getChannel().getId(), is("C02ELGNBH"));
-        assertThat(event.getChannel().getName(), is("new_name"));
+        assertThat(event.getChannel().getName(), is("channel-renamed"));
         assertThat(event.getChannel().getCreated(), is(1360782804));
     }
 

slack-api-model/src/test/java/test_locally/api/model/event/GroupDeletedEventTest.java

@@ -18,9 +18,12 @@ public void typeName() {
     @Test
     public void deserialize() {
         String json = "{\n" +
-                "    \"type\": \"group_deleted\",\n" +
-                "    \"channel\": \"G0QN9RGTT\"\n" +
-                "}\n";
+                "  \"channel\": \"G0QN9RGTT\",\n" +
+                "  \"date_deleted\": 1654752053,\n" +
+                "  \"actor_id\": \"U03E94MK0\",\n" +
+                "  \"type\": \"group_deleted\",\n" +
+                "  \"event_ts\": \"1654752053.000400\"\n" +
+                "}";
         GroupDeletedEvent event = GsonFactory.createSnakeCase().fromJson(json, GroupDeletedEvent.class);
         assertThat(event.getType(), is("group_deleted"));
         assertThat(event.getChannel(), is("G0QN9RGTT"));

slack-api-model/src/test/java/test_locally/api/model/event/GroupRenameEventTest.java

@@ -18,12 +18,16 @@ public void typeName() {
     @Test
     public void deserialize() {
         String json = "{\n" +
-                "    \"type\": \"group_rename\",\n" +
-                "    \"channel\": {\n" +
-                "        \"id\":\"G02ELGNBH\",\n" +
-                "        \"name\":\"new_name\",\n" +
-                "        \"created\":1360782804\n" +
-                "    }\n" +
+                "  \"type\": \"group_rename\",\n" +
+                "  \"channel\": {\n" +
+                "    \"id\": \"G02ELGNBH\",\n" +
+                "    \"is_channel\": true,\n" +
+                "    \"is_mpim\": false,\n" +
+                "    \"name\": \"new_name\",\n" +
+                "    \"name_normalized\": \"new_name\",\n" +
+                "    \"created\": 1360782804\n" +
+                "  },\n" +
+                "  \"event_ts\": \"1654751965.000300\"\n" +
                 "}";
         GroupRenameEvent event = GsonFactory.createSnakeCase().fromJson(json, GroupRenameEvent.class);
         assertThat(event.getType(), is("group_rename"));