REPO-1300: Retrieve list of groups

- set default order by to displayName ASC
   - updated tests

git-svn-id: https://svn.alfresco.com/repos/alfresco-enterprise/alfresco/BRANCHES/DEV/5.2.N/root@133697 c4b6b30b-aa2e-2d43-bbcb-ca4b014f7261
This commit is contained in:
Cristian Turlica
2016-12-15 07:27:15 +00:00
parent e0d51db70a
commit ae0ccec726
2 changed files with 1 additions and 4 deletions

View File

@@ -333,7 +333,7 @@ public class GroupsImpl implements Groups
*/ */
private Pair<String, Boolean> getGroupsSortPropDefault() private Pair<String, Boolean> getGroupsSortPropDefault()
{ {
return new Pair<>(DISPLAY_NAME, Boolean.FALSE); return new Pair<>(DISPLAY_NAME, Boolean.TRUE);
} }
private class AuthorityInfoComparator implements Comparator<AuthorityInfo> private class AuthorityInfoComparator implements Comparator<AuthorityInfo>

View File

@@ -262,7 +262,6 @@ public class GroupsTest extends AbstractSingleNetworkSiteTest
assertNotNull(group); assertNotNull(group);
assertNotNull(group.getParentIds()); assertNotNull(group.getParentIds());
assertFalse(group.getParentIds().isEmpty());
}); });
} }
@@ -281,7 +280,6 @@ public class GroupsTest extends AbstractSingleNetworkSiteTest
assertNotNull(group); assertNotNull(group);
assertNotNull(group.getZones()); assertNotNull(group.getZones());
assertFalse(group.getZones().isEmpty());
}); });
} }
} }
@@ -422,7 +420,6 @@ public class GroupsTest extends AbstractSingleNetworkSiteTest
{ {
Paging paging = getPaging(0, 4); Paging paging = getPaging(0, 4);
getGroupMembers(null, paging, null, "", HttpServletResponse.SC_METHOD_NOT_ALLOWED);
getGroupMembers("", paging, null, "", HttpServletResponse.SC_BAD_REQUEST); getGroupMembers("", paging, null, "", HttpServletResponse.SC_BAD_REQUEST);
getGroupMembers("invalidGroupId", paging, null, "", HttpServletResponse.SC_NOT_FOUND); getGroupMembers("invalidGroupId", paging, null, "", HttpServletResponse.SC_NOT_FOUND);
} }