fix(rest): make group query in check auth resource unlimited

related to CAM-11148
meiti · Dec 9, 2019 · 1a8bc04 · 1a8bc04
1 parent 1552648
commit 1a8bc04
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 3 deletions.
diff --git a/...ine-rest/src/main/java/org/camunda/bpm/engine/rest/impl/AuthorizationRestServiceImpl.java b/...ine-rest/src/main/java/org/camunda/bpm/engine/rest/impl/AuthorizationRestServiceImpl.java
@@ -204,11 +204,15 @@ protected IdentityService getIdentityService() {
   }
 
   protected List<String> getUserGroups(String userId) {
-    List<String> groupIds= new ArrayList<>();
-    List<Group> userGroups = getIdentityService().createGroupQuery().groupMember(userId).list();
+    List<Group> userGroups = getIdentityService().createGroupQuery()
+        .groupMember(userId)
+        .unlimitedList();
+
+    List<String> groupIds = new ArrayList<>();
     for (Group group : userGroups) {
       groupIds.add(group.getId());
     }
+
     return groupIds;
   }
 

diff --git a/...st/src/test/java/org/camunda/bpm/engine/rest/AuthorizationRestServiceInteractionTest.java b/...st/src/test/java/org/camunda/bpm/engine/rest/AuthorizationRestServiceInteractionTest.java
@@ -1109,7 +1109,7 @@ protected List<String> setupGroupQueryMock() {
     List<Group> groupMocks = MockProvider.createMockGroups();
     when(identityServiceMock.createGroupQuery()).thenReturn(mockGroupQuery);
     when(mockGroupQuery.groupMember(anyString())).thenReturn(mockGroupQuery);
-    when(mockGroupQuery.list()).thenReturn(groupMocks);
+    when(mockGroupQuery.unlimitedList()).thenReturn(groupMocks);
     List<String> groupIds = new ArrayList<String>();
     for (Group group : groupMocks) {
       groupIds.add(group.getId());