Skip to content

Commit 9404383

Browse files
committed
Run all the integration tests - 2023-10-03 PT
1 parent 428b535 commit 9404383

File tree

4 files changed

+79
-12
lines changed

4 files changed

+79
-12
lines changed

json-logs/samples/api/files.completeUploadExternal.json

Lines changed: 74 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,80 @@
33
"files": [
44
{
55
"id": "F00000000",
6-
"title": ""
6+
"title": "",
7+
"created": 12345,
8+
"timestamp": 12345,
9+
"name": "",
10+
"mimetype": "",
11+
"filetype": "",
12+
"pretty_type": "",
13+
"user": "U00000000",
14+
"user_team": "T00000000",
15+
"editable": false,
16+
"size": 12345,
17+
"mode": "",
18+
"is_external": false,
19+
"external_type": "",
20+
"is_public": false,
21+
"public_url_shared": false,
22+
"display_as_bot": false,
23+
"username": "",
24+
"url_private": "https://www.example.com/",
25+
"url_private_download": "https://www.example.com/",
26+
"permalink": "https://www.example.com/",
27+
"permalink_public": "https://www.example.com/",
28+
"edit_link": "https://www.example.com/",
29+
"preview": "",
30+
"preview_highlight": "",
31+
"lines": 12345,
32+
"lines_more": 12345,
33+
"preview_is_truncated": false,
34+
"comments_count": 12345,
35+
"is_starred": false,
36+
"shares": {
37+
"public": {
38+
"C00000000": [
39+
{
40+
"reply_users": [
41+
""
42+
],
43+
"reply_users_count": 12345,
44+
"reply_count": 12345,
45+
"ts": "0000000000.000000",
46+
"channel_name": "",
47+
"team_id": "T00000000",
48+
"share_user_id": "U00000000"
49+
}
50+
],
51+
"C00000001": [
52+
{
53+
"reply_users": [
54+
""
55+
],
56+
"reply_users_count": 12345,
57+
"reply_count": 12345,
58+
"ts": "0000000000.000000",
59+
"channel_name": "",
60+
"team_id": "T00000000",
61+
"share_user_id": "U00000000"
62+
}
63+
]
64+
}
65+
},
66+
"channels": [
67+
"C00000000",
68+
""
69+
],
70+
"groups": [
71+
""
72+
],
73+
"ims": [
74+
""
75+
],
76+
"has_more_shares": false,
77+
"has_rich_preview": false,
78+
"file_access": "",
79+
"media_display_type": ""
780
}
881
],
982
"error": "",

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

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -75,7 +75,8 @@
7575
"is_admin": false,
7676
"is_owner": false,
7777
"teams": [
78-
""
78+
"",
79+
"T00000000"
7980
],
8081
"is_primary_owner": false
8182
},

slack-api-client/src/main/java/com/slack/api/methods/impl/FilesUploadV2Helper.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -106,7 +106,7 @@ public FilesUploadV2Response completeUploads(
106106
}
107107

108108
result.setFiles(new ArrayList<>());
109-
for (FilesCompleteUploadExternalResponse.FileDetails file : response.getFiles()) {
109+
for (File file : response.getFiles()) {
110110
if (v2Request.isRequestFileInfo()) {
111111
FilesInfoResponse fileInfo = this.client.filesInfo(r -> r.token(v2Request.getToken()).file(file.getId()));
112112
underlyingException.getFileInfoResponses().add(fileInfo);
Lines changed: 2 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,7 @@
11
package com.slack.api.methods.response.files;
22

33
import com.slack.api.methods.SlackApiTextResponse;
4+
import com.slack.api.model.File;
45
import com.slack.api.model.ResponseMetadata;
56
import lombok.Data;
67

@@ -16,14 +17,6 @@ public class FilesCompleteUploadExternalResponse implements SlackApiTextResponse
1617
private String needed;
1718
private String provided;
1819
private transient Map<String, List<String>> httpResponseHeaders;
19-
20-
private List<FileDetails> files;
21-
22-
@Data
23-
public static class FileDetails {
24-
private String id;
25-
private String title;
26-
}
27-
20+
private List<File> files;
2821
private ResponseMetadata responseMetadata;
2922
}

0 commit comments

Comments
 (0)