Skip to content

Commit 7f69f8a

Browse files
committed
Run all the integration tests - 2021-09-07 PT
1 parent 7a55ca1 commit 7f69f8a

File tree

10 files changed

+60
-6
lines changed

10 files changed

+60
-6
lines changed

json-logs/raw/audit/v1/actions.json

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -137,7 +137,11 @@
137137
"user_force_upgrade_non_compliant_mobile_app_version",
138138
"user_login_failed",
139139
"user_session_invalidated",
140-
"user_session_settings_changed"
140+
"user_session_settings_changed",
141+
"connect_dm_invite_generated",
142+
"connect_dm_invite_revoked",
143+
"connect_dm_invite_accepted",
144+
"connect_dm_invite_ignored"
141145
],
142146
"file": [
143147
"file_downloaded",

json-logs/samples/api/reminders.add.json

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -10,6 +10,11 @@
1010
"complete_ts": 12345
1111
},
1212
"error": "",
13+
"response_metadata": {
14+
"messages": [
15+
""
16+
]
17+
},
1318
"needed": "",
1419
"provided": ""
1520
}

json-logs/samples/api/reminders.list.json

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,13 @@
99
"recurring": false,
1010
"time": 12345,
1111
"complete_ts": 12345,
12-
"channel": "C00000000"
12+
"channel": "C00000000",
13+
"recurrence": {
14+
"frequency": "",
15+
"weekdays": [
16+
""
17+
]
18+
}
1319
}
1420
],
1521
"error": "",

json-logs/samples/audit/v1/logs.json

Lines changed: 9 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -202,7 +202,15 @@
202202
"external_user_email": "",
203203
"channel_id": "",
204204
"added_team_id": "",
205-
"is_token_rotation_enabled_app": ""
205+
"is_token_rotation_enabled_app": "",
206+
"old_retention_policy": {
207+
"type": "",
208+
"duration_days": 123
209+
},
210+
"new_retention_policy": {
211+
"type": "",
212+
"duration_days": 123
213+
}
206214
}
207215
}
208216
]

slack-api-client/src/main/java/com/slack/api/audit/Actions.java

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -153,6 +153,10 @@ private User() {
153153
public static final String user_session_reset_by_admin = "user_session_reset_by_admin";
154154
public static final String user_logout_non_compliant_mobile_app_version = "user_logout_non_compliant_mobile_app_version";
155155
public static final String user_force_upgrade_non_compliant_mobile_app_version = "user_force_upgrade_non_compliant_mobile_app_version";
156+
public static final String connect_dm_invite_generated = "connect_dm_invite_generated";
157+
public static final String connect_dm_invite_revoked = "connect_dm_invite_revoked";
158+
public static final String connect_dm_invite_accepted = "connect_dm_invite_accepted";
159+
public static final String connect_dm_invite_ignored = "connect_dm_invite_ignored";
156160
}
157161

158162
public static class File {

slack-api-client/src/main/java/com/slack/api/audit/response/LogsResponse.java

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -217,6 +217,8 @@ public static class Details {
217217
private String channelId; // connect_dm_invite_accepted
218218
private String addedTeamId; // approved_orgs_added
219219
private String isTokenRotationEnabledApp; // app_scopes_expanded
220+
private MessageRetentionPolicy oldRetentionPolicy; // channel_retention_changed
221+
private MessageRetentionPolicy newRetentionPolicy; // channel_retention_changed
220222
}
221223

222224
@Data
@@ -267,4 +269,10 @@ public static class WildCard {
267269
private String type;
268270
}
269271

272+
@Data
273+
public static class MessageRetentionPolicy {
274+
private String type;
275+
private Integer durationDays;
276+
}
277+
270278
}

slack-api-client/src/main/java/com/slack/api/methods/response/reminders/RemindersAddResponse.java

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,7 @@
22

33
import com.slack.api.methods.SlackApiTextResponse;
44
import com.slack.api.model.Reminder;
5+
import com.slack.api.model.ResponseMetadata;
56
import lombok.Data;
67

78
@Data
@@ -14,4 +15,5 @@ public class RemindersAddResponse implements SlackApiTextResponse {
1415
private String provided;
1516

1617
private Reminder reminder;
18+
private ResponseMetadata responseMetadata;
1719
}

slack-api-client/src/test/java/test_locally/api/methods/FieldValidation_o_to_z_Test.java

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -143,8 +143,8 @@ public void reactions_remove() throws Exception {
143143
@Test
144144
public void reminders_add() throws Exception {
145145
RemindersAddResponse obj = parse("reminders.add", RemindersAddResponse.class);
146-
verifyIfAllGettersReturnNonNull(obj);
147-
verifyIfAllGettersReturnNonNullRecursively(obj.getReminder(), "getChannel");
146+
verifyIfAllGettersReturnNonNull(obj, "getWarning");
147+
verifyIfAllGettersReturnNonNullRecursively(obj.getReminder(), "getRecurrence", "getChannel", "getItem");
148148
}
149149

150150
@Test
@@ -163,7 +163,7 @@ public void reminders_delete() throws Exception {
163163
public void reminders_info() throws Exception {
164164
RemindersInfoResponse obj = parse("reminders.info", RemindersInfoResponse.class);
165165
verifyIfAllGettersReturnNonNull(obj);
166-
verifyIfAllGettersReturnNonNullRecursively(obj.getReminder(), "getChannel");
166+
verifyIfAllGettersReturnNonNullRecursively(obj.getReminder(), "getChannel", "getRecurrence", "getItem");
167167
}
168168

169169
@Test

slack-api-client/src/test/java/test_with_remote_apis/methods/reminders_Test.java

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2,12 +2,15 @@
22

33
import com.slack.api.Slack;
44
import com.slack.api.methods.response.reminders.*;
5+
import com.slack.api.model.Reminder;
56
import config.Constants;
67
import config.SlackTestConfig;
78
import lombok.extern.slf4j.Slf4j;
89
import org.junit.AfterClass;
910
import org.junit.Test;
1011

12+
import java.util.Arrays;
13+
1114
import static org.hamcrest.CoreMatchers.is;
1215
import static org.hamcrest.CoreMatchers.nullValue;
1316
import static org.hamcrest.MatcherAssert.assertThat;

slack-api-model/src/main/java/com/slack/api/model/Reminder.java

Lines changed: 14 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,8 @@
55
import lombok.Data;
66
import lombok.NoArgsConstructor;
77

8+
import java.util.List;
9+
810
/**
911
* https://api.slack.com/methods/reminders.add
1012
*/
@@ -22,4 +24,16 @@ public class Reminder {
2224
private boolean recurring;
2325
private Integer time;
2426
private Integer completeTs;
27+
private Recurrence recurrence;
28+
// TODO: finalize the type
29+
private Object item;
30+
31+
@Data
32+
@Builder
33+
@NoArgsConstructor
34+
@AllArgsConstructor
35+
public static class Recurrence {
36+
private String frequency;
37+
private List<String> weekdays;
38+
}
2539
}

0 commit comments

Comments
 (0)