inteligr8/alfresco-community-repo
1
0
Fork 0
mirror of https://github.com/Alfresco/alfresco-community-repo.git synced 2025-08-14 17:58:59 +00:00
Code Issues Packages Projects Releases Wiki Activity

Merged 5.2.N (5.2.1) to HEAD (5.2)

Browse Source 
130614 jvonka: REPO-1059: REST API: Invalid orderBy field is ignored
   - minor follow-on (comment in the specific test when this issue was found - in addition to the new generic tests)


git-svn-id: https://svn.alfresco.com/repos/alfresco-enterprise/alfresco/HEAD/root@132154 c4b6b30b-aa2e-2d43-bbcb-ca4b014f7261
This commit is contained in:
Alan Davis
2016-11-03 13:13:03 +00:00
parent 07fe704500
commit 9e82c8a568
1 changed files with 9 additions and 9 deletions
Download Patch File Download Diff File Expand all files Collapse all files

18
source/test-java/org/alfresco/rest/api/tests/QueriesPeopleApiTest.java
View File

@@ -511,15 +511,15 @@ public class QueriesPeopleApiTest extends AbstractSingleNetworkSiteTest
checkApiCall(term, orderBy, fields, paging, expectedStatus, expectedPeople); checkApiCall(term, orderBy, fields, paging, expectedStatus, expectedPeople);
} }
// // TODO Rubbish is taken to be "asc" so returns 200 rather than 400 - is this a framework bug? @Test
// @Test public void testBadOrderByDirection() throws Exception
// public void testBadOrderByDirection() throws Exception {
// { // note: also tested generically in RecognizedParamsExtractorTest
// orderBy = "firstName rubbish, lastName asc"; orderBy = "firstName rubbish, lastName asc";
// expectedStatus = 400; expectedStatus = 400;
//
// checkApiCall(term, orderBy, fields, paging, expectedStatus, expectedPeople); checkApiCall(term, orderBy, fields, paging, expectedStatus, expectedPeople);
// } }
// // TODO Having a space in the list discards everything after the space - found in manual testing - is this a framework bug? // // TODO Having a space in the list discards everything after the space - found in manual testing - is this a framework bug?
// @Test // @Test