From 44f9d72d42d4e78f6d2abdb8bd24bde607d03ca4 Mon Sep 17 00:00:00 2001 From: Jan Vonka Date: Mon, 12 Sep 2016 13:14:00 +0000 Subject: [PATCH] REPO-1059: REST API: Invalid orderBy field direction is ignored (hence defaults to ascending rather than returning an error) - follow-on to r13055 to fix test fallout git-svn-id: https://svn.alfresco.com/repos/alfresco-enterprise/alfresco/BRANCHES/DEV/5.2.N/root@130573 c4b6b30b-aa2e-2d43-bbcb-ca4b014f7261 --- .../api/tests/ProcessWorkflowApiTest.java | 23 +++++++++++-------- 1 file changed, 13 insertions(+), 10 deletions(-) diff --git a/source/test-java/org/alfresco/rest/workflow/api/tests/ProcessWorkflowApiTest.java b/source/test-java/org/alfresco/rest/workflow/api/tests/ProcessWorkflowApiTest.java index 33cb38be61..4bfc5dc8c7 100644 --- a/source/test-java/org/alfresco/rest/workflow/api/tests/ProcessWorkflowApiTest.java +++ b/source/test-java/org/alfresco/rest/workflow/api/tests/ProcessWorkflowApiTest.java @@ -1062,16 +1062,19 @@ public class ProcessWorkflowApiTest extends EnterpriseWorkflowTestApi { assertEquals(HttpStatus.BAD_REQUEST.value(), e.getHttpResponse().getStatusCode()); } - - // sort on non existing sort order (default ASC is taken) - paramMap.put("orderBy", "businessKey ASC2"); - processList = processesClient.getProcesses(paramMap); - assertNotNull(processList); - assertEquals(3, processList.getList().size()); - - assertEquals(process3.getId(), processList.getList().get(0).getId()); - assertEquals(process1.getId(), processList.getList().get(1).getId()); - assertEquals(process2.getId(), processList.getList().get(2).getId()); + + // sort on non existing sort order + try + { + + paramMap.put("orderBy", "businessKey ASC2"); + processList = processesClient.getProcesses(paramMap); + fail("forbidden expected"); + } + catch (PublicApiException e) + { + assertEquals(HttpStatus.BAD_REQUEST.value(), e.getHttpResponse().getStatusCode()); + } } finally {