Skip to content

Commit ee4c798

Browse files
korzolkorzol
committed
feat: Support for multivalued status as described in issue #315.
1 parent 027181e commit ee4c798

4 files changed

Lines changed: 158 additions & 27 deletions

File tree

‎src/main/java/com/crowdin/client/tasks/TasksApi.java‎

Lines changed: 54 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -14,9 +14,11 @@
1414
import com.crowdin.client.core.model.ResponseObject;
1515
import com.crowdin.client.tasks.model.*;
1616

17+
import java.util.EnumSet;
1718
import java.util.List;
1819
import java.util.Map;
1920
import java.util.Optional;
21+
import java.util.stream.Collectors;
2022

2123
public class TasksApi extends CrowdinApi {
2224
public TasksApi(Credentials credentials) {
@@ -40,14 +42,30 @@ public TasksApi(Credentials credentials, ClientConfig clientConfig) {
4042
* </ul>
4143
*/
4244
public ResponseList<Task> listTasks(Long projectId, Integer limit, Integer offset, Status status, Integer assigneeId) throws HttpException, HttpBadRequestException {
43-
Map<String, Optional<Object>> queryParams = HttpRequestConfig.buildUrlParams(
44-
"status", Optional.ofNullable(status),
45-
"assigneeId", Optional.ofNullable(assigneeId),
46-
"limit", Optional.ofNullable(limit),
47-
"offset", Optional.ofNullable(offset)
48-
);
49-
TaskResponseList taskResponseList = this.httpClient.get(this.url + "/projects/" + projectId + "/tasks", new HttpRequestConfig(queryParams), TaskResponseList.class);
50-
return TaskResponseList.to(taskResponseList);
45+
EnumSet<Status> statuses = (status != null) ? EnumSet.of(status) : null;
46+
47+
ListTasksParams params = new ListTasksParams();
48+
params.setStatuses(statuses);
49+
params.setLimit(limit);
50+
params.setOffset(offset);
51+
params.setAssigneeId(assigneeId);
52+
53+
return listTasksInternal(projectId, params);
54+
}
55+
56+
/**
57+
* Lists tasks for a given project, filtered by multiple statuses.
58+
*
59+
* @param projectId ID of the project
60+
* @param params Query params
61+
* @return List of tasks
62+
* @see <ul>
63+
* <li><a href="https://developer.crowdin.com/api/v2/#operation/api.projects.tasks.getMany" target="_blank"><b>API Documentation</b></a></li>
64+
* <li><a href="https://developer.crowdin.com/enterprise/api/v2/#operation/api.projects.tasks.getMany" target="_blank"><b>Enterprise API Documentation</b></a></li>
65+
* </ul>
66+
*/
67+
public ResponseList<Task> listTasks(Long projectId, ListTasksParams params) throws HttpException, HttpBadRequestException {
68+
return listTasksInternal(projectId, params);
5169
}
5270

5371
/**
@@ -159,6 +177,34 @@ public ResponseObject<Task> editTaskArchivedStatus(Long taskId, Long projectId,
159177
return ResponseObject.of(taskResponseObject.getData());
160178
}
161179

180+
//<editor-fold desc="List Tasks Common Method">
181+
private ResponseList<Task> listTasksInternal(Long projectId, ListTasksParams params) throws HttpException, HttpBadRequestException {
182+
ListTasksParams query = Optional.ofNullable(params).orElse(new ListTasksParams());
183+
184+
EnumSet<Status> statuses = query.getStatuses();
185+
186+
Map<String, Optional<Object>> queryParams = HttpRequestConfig.buildUrlParams(
187+
"status", Optional.ofNullable(
188+
statuses == null ? null : statuses.stream()
189+
.map(status -> status.to(status))
190+
.collect(Collectors.joining(","))
191+
),
192+
"assigneeId", Optional.ofNullable(query.getAssigneeId()),
193+
"limit", Optional.ofNullable(query.getLimit()),
194+
"offset", Optional.ofNullable(query.getOffset())
195+
);
196+
197+
TaskResponseList taskResponseList = this.httpClient.get(
198+
this.url + "/projects/" + projectId + "/tasks",
199+
new HttpRequestConfig(queryParams),
200+
TaskResponseList.class
201+
);
202+
203+
return TaskResponseList.to(taskResponseList);
204+
}
205+
206+
//</editor-fold>
207+
162208
//<editor-fold desc="Task Settings Templates">
163209

164210
/**

‎src/main/java/com/crowdin/client/tasks/model/ListTasksParams.java‎

Lines changed: 14 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,14 @@
1+
package com.crowdin.client.tasks.model;
2+
3+
import com.crowdin.client.core.model.Pagination;
4+
import lombok.Data;
5+
import lombok.EqualsAndHashCode;
6+
7+
import java.util.EnumSet;
8+
9+
@EqualsAndHashCode(callSuper = true)
10+
@Data
11+
public class ListTasksParams extends Pagination {
12+
private EnumSet<Status> statuses;
13+
private Integer assigneeId;
14+
}

‎src/test/java/com/crowdin/client/tasks/TasksApiTest.java‎

Lines changed: 35 additions & 19 deletions
Original file line numberDiff line numberDiff line change
@@ -7,17 +7,7 @@
77
import com.crowdin.client.core.model.ResponseObject;
88
import com.crowdin.client.framework.RequestMock;
99
import com.crowdin.client.framework.TestClient;
10-
import com.crowdin.client.tasks.model.AssignedTeam;
11-
import com.crowdin.client.tasks.model.Assignee;
12-
import com.crowdin.client.tasks.model.AssigneeRequest;
13-
import com.crowdin.client.tasks.model.CreateTaskEnterpriseStringsBasedRequest;
14-
import com.crowdin.client.tasks.model.CreateTaskRequest;
15-
import com.crowdin.client.tasks.model.CreateTaskEnterpriseVendorRequest;
16-
import com.crowdin.client.tasks.model.CreateTaskStringsBasedRequest;
17-
import com.crowdin.client.tasks.model.Progress;
18-
import com.crowdin.client.tasks.model.Status;
19-
import com.crowdin.client.tasks.model.Task;
20-
import com.crowdin.client.tasks.model.Type;
10+
import com.crowdin.client.tasks.model.*;
2111
import com.crowdin.client.tasks.model.pending.CreateEnterprisePendingTaskRequest;
2212
import com.crowdin.client.tasks.model.pending.CreatePendingTaskRequest;
2313
import org.apache.http.client.methods.HttpDelete;
@@ -26,21 +16,17 @@
2616
import org.apache.http.client.methods.HttpPost;
2717
import org.junit.jupiter.api.Test;
2818

29-
import java.util.Arrays;
30-
import java.util.Calendar;
31-
import java.util.Date;
32-
import java.util.HashMap;
33-
import java.util.List;
34-
import java.util.Map;
35-
import java.util.TimeZone;
19+
import java.util.*;
3620

3721
import static java.util.Collections.singletonList;
3822
import static org.junit.jupiter.api.Assertions.assertEquals;
23+
import static org.junit.jupiter.api.Assertions.assertTrue;
3924

4025
public class TasksApiTest extends TestClient {
4126

4227
private final Long projectId = 12L;
4328
private final Long enterpriseProjectId = 13L;
29+
private final Long multiStatusProjectId = 14L;
4430
private final Long taskId = 2L;
4531
private final Long prevTaskId = 1L;
4632
private final Status status = Status.TODO;
@@ -61,7 +47,8 @@ public List<RequestMock> getMocks() {
6147
RequestMock.build(this.url + "/projects/" + projectId + "/tasks/" + taskId, HttpDelete.METHOD_NAME),
6248
RequestMock.build(this.url + "/projects/" + projectId + "/tasks/" + taskId, HttpPatch.METHOD_NAME, "api/tasks/editTask.json", "api/tasks/task.json"),
6349
RequestMock.build(this.url + "/user/tasks", HttpGet.METHOD_NAME, "api/tasks/listTasks.json"),
64-
RequestMock.build(this.url + "/user/tasks/" + taskId, HttpPatch.METHOD_NAME, "api/tasks/editTask.json", "api/tasks/task.json")
50+
RequestMock.build(this.url + "/user/tasks/" + taskId, HttpPatch.METHOD_NAME, "api/tasks/editTask.json", "api/tasks/task.json"),
51+
RequestMock.build(this.url + "/projects/" + multiStatusProjectId + "/tasks", HttpGet.METHOD_NAME, "api/tasks/multiStatusListTasks.json", Map.of("status", "todo,done"))
6552
);
6653
}
6754

@@ -83,6 +70,35 @@ public void listTasksTest() {
8370
assertEquals(taskResponseList.getData().get(0).getData().getAssignees().get(0), assignee);
8471
}
8572

73+
@Test
74+
public void listTasksTest_multipleStatuses() {
75+
Assignee assignee = new Assignee();
76+
assignee.setId(1L);
77+
assignee.setUsername("john_smith");
78+
assignee.setFullName("john_smith");
79+
assignee.setAvatarUrl("");
80+
assignee.setWordsCount(5);
81+
assignee.setWordsLeft(3);
82+
TimeZone.setDefault(TimeZone.getTimeZone("GMT"));
83+
84+
EnumSet<Status> statuses = EnumSet.of(Status.TODO, Status.DONE);
85+
86+
ListTasksParams listTasksParams = new ListTasksParams();
87+
listTasksParams.setStatuses(statuses);
88+
89+
ResponseList<Task> taskResponseList = this.getTasksApi().listTasks(multiStatusProjectId, listTasksParams);
90+
91+
assertEquals(1, taskResponseList.getData().size());
92+
assertEquals(taskId, taskResponseList.getData().get(0).getData().getId());
93+
assertEquals(multiStatusProjectId, taskResponseList.getData().get(0).getData().getProjectId());
94+
95+
Status responseProjectStatus = taskResponseList.getData().get(0).getData().getStatus();
96+
assertTrue(statuses.contains(responseProjectStatus));
97+
98+
assertEquals(new Date(119, Calendar.SEPTEMBER,27,7,0,14), taskResponseList.getData().get(0).getData().getDeadline());
99+
assertEquals(taskResponseList.getData().get(0).getData().getAssignees().get(0), assignee);
100+
}
101+
86102
@Test
87103
public void addTaskTest() {
88104
CreateTaskRequest request = new CreateTaskRequest();

‎src/test/resources/api/tasks/multiStatusListTasks.json‎

Lines changed: 55 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,55 @@
1+
{
2+
"data": [
3+
{
4+
"data": {
5+
"id": 2,
6+
"projectId": 14,
7+
"creatorId": 6,
8+
"type": 1,
9+
"status": "todo",
10+
"title": "French",
11+
"assignees": [
12+
{
13+
"id": 1,
14+
"username": "john_smith",
15+
"fullName": "john_smith",
16+
"avatarUrl": "",
17+
"wordsCount": 5,
18+
"wordsLeft": 3
19+
}
20+
],
21+
"assignedTeams": [
22+
{
23+
"id": 1,
24+
"wordsCount": 5
25+
}
26+
],
27+
"fileIds": [
28+
1
29+
],
30+
"progress": {
31+
"total": 24,
32+
"done": 15,
33+
"percent": 62
34+
},
35+
"sourceLanguageId": "en",
36+
"targetLanguageId": "fr",
37+
"description": "Proofread all French strings",
38+
"hash": "dac37aff364d83899128e68afe0de4994",
39+
"translationUrl": "/proofread/9092638ac9f2a2d1b5571d08edc53763/all/en-fr/10?task=dac37aff364d83899128e68afe0de4994",
40+
"wordsCount": 24,
41+
"filesCount": 2,
42+
"commentsCount": 0,
43+
"deadline": "2019-09-27T07:00:14+00:00",
44+
"timeRange": "string",
45+
"workflowStepId": 10,
46+
"createdAt": "2019-09-23T09:04:29+00:00",
47+
"updatedAt": "2019-09-23T09:04:29+00:00"
48+
}
49+
}
50+
],
51+
"pagination": {
52+
"offset": 0,
53+
"limit": 25
54+
}
55+
}

0 commit comments

Comments
 (0)