TSK-889: Replace WorkbasketAccessItemListResource with equivalent PaginatedListResource

This commit is contained in:
Benjamin Eckstein 2020-01-08 14:14:02 +01:00 committed by Holger Hagen
parent 479faed8de
commit f06f87f41e
7 changed files with 47 additions and 45 deletions

View File

@ -41,7 +41,7 @@ import pro.taskana.rest.resource.DistributionTargetListResource;
import pro.taskana.rest.resource.DistributionTargetResource;
import pro.taskana.rest.resource.DistributionTargetResourceAssembler;
import pro.taskana.rest.resource.PagedResources.PageMetadata;
import pro.taskana.rest.resource.WorkbasketAccessItemListResource;
import pro.taskana.rest.resource.WorkbasketAccessItemPaginatedListResource;
import pro.taskana.rest.resource.WorkbasketAccessItemResource;
import pro.taskana.rest.resource.WorkbasketAccessItemResourceAssembler;
import pro.taskana.rest.resource.WorkbasketResource;
@ -201,11 +201,11 @@ public class WorkbasketController extends AbstractPagingController {
@GetMapping(path = Mapping.URL_WORKBASKET_ID_ACCESSITEMS)
@Transactional(readOnly = true, rollbackFor = Exception.class)
public ResponseEntity<WorkbasketAccessItemListResource> getWorkbasketAccessItems(
public ResponseEntity<WorkbasketAccessItemPaginatedListResource> getWorkbasketAccessItems(
@PathVariable(value = "workbasketId") String workbasketId)
throws NotAuthorizedException, WorkbasketNotFoundException {
LOGGER.debug("Entry to getWorkbasketAccessItems(workbasketId= {})", workbasketId);
ResponseEntity<WorkbasketAccessItemListResource> result;
ResponseEntity<WorkbasketAccessItemPaginatedListResource> result;
List<WorkbasketAccessItem> accessItems =
workbasketService.getWorkbasketAccessItems(workbasketId);
@ -221,7 +221,7 @@ public class WorkbasketController extends AbstractPagingController {
@PutMapping(path = Mapping.URL_WORKBASKET_ID_ACCESSITEMS)
@Transactional(rollbackFor = Exception.class)
public ResponseEntity<WorkbasketAccessItemListResource> setWorkbasketAccessItems(
public ResponseEntity<WorkbasketAccessItemPaginatedListResource> setWorkbasketAccessItems(
@PathVariable(value = "workbasketId") String workbasketId,
@RequestBody List<WorkbasketAccessItemResource> workbasketAccessResourceItems)
throws NotAuthorizedException, InvalidArgumentException, WorkbasketNotFoundException {
@ -237,7 +237,7 @@ public class WorkbasketController extends AbstractPagingController {
List<WorkbasketAccessItem> updatedWbAccessItems =
workbasketService.getWorkbasketAccessItems(workbasketId);
ResponseEntity<WorkbasketAccessItemListResource> response =
ResponseEntity<WorkbasketAccessItemPaginatedListResource> response =
ResponseEntity.ok(
workbasketAccessItemResourceAssembler.toResources(workbasketId, updatedWbAccessItems));
if (LOGGER.isDebugEnabled()) {

View File

@ -1,29 +0,0 @@
package pro.taskana.rest.resource;
import com.fasterxml.jackson.annotation.JsonIgnore;
import java.util.Collection;
import org.springframework.hateoas.Link;
/** Resource class for {@link WorkbasketAccessItemResource} without Pagination. */
public class WorkbasketAccessItemListResource extends WorkbasketAccessItemPaginatedListResource {
public WorkbasketAccessItemListResource() {
super();
}
public WorkbasketAccessItemListResource(
Collection<WorkbasketAccessItemResource> content, Link... links) {
super(content, null, links);
}
public WorkbasketAccessItemListResource(
Collection<WorkbasketAccessItemResource> content, Iterable<Link> links) {
super(content, null, links);
}
@Override
@JsonIgnore
public PageMetadata getMetadata() {
return super.getMetadata();
}
}

View File

@ -2,6 +2,7 @@ package pro.taskana.rest.resource;
import com.fasterxml.jackson.annotation.JsonProperty;
import java.util.Collection;
import java.util.Objects;
import org.springframework.hateoas.Link;
/** Resource class for {@link WorkbasketAccessItemResource} with Pagination. */
@ -29,4 +30,14 @@ public class WorkbasketAccessItemPaginatedListResource
public Collection<WorkbasketAccessItemResource> getContent() {
return super.getContent();
}
@Override
public PageMetadata getMetadata() {
PageMetadata pageMetadata = super.getMetadata();
if (Objects.isNull(pageMetadata)) {
Collection<WorkbasketAccessItemResource> content = getContent();
return new PageMetadata(content.size(), 0, content.size());
}
return pageMetadata;
}
}

View File

@ -54,11 +54,11 @@ public class WorkbasketAccessItemResourceAssembler
return new WorkbasketAccessItemPaginatedListResource(toResources(entities), pageMetadata);
}
public WorkbasketAccessItemListResource toResources(
public WorkbasketAccessItemPaginatedListResource toResources(
String workbasketId, List<WorkbasketAccessItem> entities)
throws NotAuthorizedException, WorkbasketNotFoundException {
WorkbasketAccessItemListResource accessItemListResource =
new WorkbasketAccessItemListResource(super.toResources(entities));
WorkbasketAccessItemPaginatedListResource accessItemListResource =
new WorkbasketAccessItemPaginatedListResource(super.toResources(entities), null);
accessItemListResource.add(
linkTo(methodOn(WorkbasketController.class).getWorkbasketAccessItems(workbasketId))
.withSelfRel());

View File

@ -56,7 +56,10 @@ class WorkbasketAccessItemControllerRestDocumentation extends BaseRestDocumentat
accessItemFieldDescriptionsMap.put("accessItems.permCustom11", "");
accessItemFieldDescriptionsMap.put("accessItems.permCustom12", "");
accessItemFieldDescriptionsMap.put("_links.self.href", "Link to self");
accessItemFieldDescriptionsMap.put("page", "Number of page");
accessItemFieldDescriptionsMap.put("page.size", "The page size");
accessItemFieldDescriptionsMap.put("page.number", "Zero-indexed current page");
accessItemFieldDescriptionsMap.put("page.totalElements", "The total number of elements");
accessItemFieldDescriptionsMap.put("page.totalPages", "Total amount of pages");
accessItemFieldDescriptors =
new FieldDescriptor[] {
@ -106,7 +109,13 @@ class WorkbasketAccessItemControllerRestDocumentation extends BaseRestDocumentat
.description(accessItemFieldDescriptionsMap.get("accessItems.permCustom12")),
fieldWithPath("_links.self.href")
.description(accessItemFieldDescriptionsMap.get("_links.self.href")),
fieldWithPath("page").description(accessItemFieldDescriptionsMap.get("page"))
fieldWithPath("page.size").description(accessItemFieldDescriptionsMap.get("page.size")),
fieldWithPath("page.number")
.description(accessItemFieldDescriptionsMap.get("page.number")),
fieldWithPath("page.totalElements")
.description(accessItemFieldDescriptionsMap.get("page.totalElements")),
fieldWithPath("page.totalPages")
.description(accessItemFieldDescriptionsMap.get("page.totalPages")),
};
}

View File

@ -109,6 +109,10 @@ class WorkbasketControllerRestDocumentation extends BaseRestDocumentation {
accessItemFieldDescriptionsMap.put("accessItems.permCustom12", "");
accessItemFieldDescriptionsMap.put(
"accessItems._links.workbasket.href", "Link to the workbasket");
accessItemFieldDescriptionsMap.put("page.size", "The page size");
accessItemFieldDescriptionsMap.put("page.number", "Zero-indexed current page");
accessItemFieldDescriptionsMap.put("page.totalElements", "The total number of elements");
accessItemFieldDescriptionsMap.put("page.totalPages", "Total amount of pages");
allWorkbasketsFieldDescriptors =
new FieldDescriptor[] {
@ -225,6 +229,13 @@ class WorkbasketControllerRestDocumentation extends BaseRestDocumentation {
.description(accessItemFieldDescriptionsMap.get("accessItems.permCustom11")),
fieldWithPath("accessItems[].permCustom12")
.description(accessItemFieldDescriptionsMap.get("accessItems.permCustom12")),
fieldWithPath("page.size").description(accessItemFieldDescriptionsMap.get("page.size")),
fieldWithPath("page.number")
.description(accessItemFieldDescriptionsMap.get("page.number")),
fieldWithPath("page.totalElements")
.description(accessItemFieldDescriptionsMap.get("page.totalElements")),
fieldWithPath("page.totalPages")
.description(accessItemFieldDescriptionsMap.get("page.totalPages")),
fieldWithPath("_links.self.href").ignored(),
fieldWithPath("_links.workbasket.href").ignored()
};
@ -233,6 +244,7 @@ class WorkbasketControllerRestDocumentation extends BaseRestDocumentation {
new FieldDescriptor[] {
subsectionWithPath("accessItems")
.description("An array of <<access-item, Access Items>>"),
subsectionWithPath("page").description("Number of page"),
fieldWithPath("_links.self.href").ignored(),
fieldWithPath("_links.workbasket.href").ignored()
};

View File

@ -21,7 +21,6 @@ import org.springframework.web.client.RestTemplate;
import pro.taskana.RestHelper;
import pro.taskana.TaskanaSpringBootTest;
import pro.taskana.rest.resource.WorkbasketAccessItemListResource;
import pro.taskana.rest.resource.WorkbasketAccessItemPaginatedListResource;
/** Test WorkbasketAccessItemController. */
@ -39,24 +38,24 @@ class WorkbasketAccessItemControllerIntTest {
@Test
void testGetAllWorkbasketAccessItems() {
ResponseEntity<WorkbasketAccessItemListResource> response =
ResponseEntity<WorkbasketAccessItemPaginatedListResource> response =
template.exchange(
restHelper.toUrl(Mapping.URL_WORKBASKETACCESSITEMS),
HttpMethod.GET,
restHelper.defaultRequest(),
ParameterizedTypeReference.forType(WorkbasketAccessItemListResource.class));
ParameterizedTypeReference.forType(WorkbasketAccessItemPaginatedListResource.class));
assertNotNull(response.getBody().getLink(Link.REL_SELF));
}
@Test
void testGetWorkbasketAccessItemsKeepingFilters() {
String parameters = "?sort-by=workbasket-key&order=asc&page=1&page-size=9&access-ids=user_1_1";
ResponseEntity<WorkbasketAccessItemListResource> response =
ResponseEntity<WorkbasketAccessItemPaginatedListResource> response =
template.exchange(
restHelper.toUrl(Mapping.URL_WORKBASKETACCESSITEMS) + parameters,
HttpMethod.GET,
restHelper.defaultRequest(),
ParameterizedTypeReference.forType(WorkbasketAccessItemListResource.class));
ParameterizedTypeReference.forType(WorkbasketAccessItemPaginatedListResource.class));
assertNotNull(response.getBody().getLink(Link.REL_SELF));
assertTrue(response.getBody().getLink(Link.REL_SELF).getHref().endsWith(parameters));
}
@ -69,7 +68,7 @@ class WorkbasketAccessItemControllerIntTest {
+ "?sort-by=workbasket-key&order=asc&page=1&page-size=9&invalid=user_1_1",
HttpMethod.GET,
restHelper.defaultRequest(),
ParameterizedTypeReference.forType(WorkbasketAccessItemListResource.class));
ParameterizedTypeReference.forType(WorkbasketAccessItemPaginatedListResource.class));
fail();
} catch (HttpClientErrorException e) {
assertEquals(HttpStatus.BAD_REQUEST, e.getStatusCode());