OAK-11317 : removed conversion to set twice for ExternalGroupPrincipa…

…lProviderTest
apache · Dec 20, 2024 · b02f36f · b02f36f
1 parent 412cad0
commit b02f36f
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/...i/security/authentication/external/impl/principal/ExternalGroupPrincipalProviderTest.java b/...i/security/authentication/external/impl/principal/ExternalGroupPrincipalProviderTest.java
@@ -90,13 +90,13 @@ void sync(@NotNull ExternalUser externalUser) throws Exception {
     @NotNull
     Set<Principal> getExpectedGroupPrincipals(@NotNull String userId) throws Exception {
         if (syncConfig.user().getMembershipNestingDepth() == 1) {
-            return CollectionUtils.toSet(idp.getUser(userId).getDeclaredGroups()).stream().map(externalIdentityRef -> {
+            return CollectionUtils.toStream(idp.getUser(userId).getDeclaredGroups()).map(externalIdentityRef -> {
                 try {
                     return new PrincipalImpl(idp.getIdentity(externalIdentityRef).getPrincipalName());
                 } catch (ExternalIdentityException e) {
                     throw new RuntimeException(e);
                 }
-            }).collect(Collectors.toUnmodifiableSet());
+            }).collect(Collectors.toSet());
         } else {
             Set<Principal> set = new HashSet<>();
             collectExpectedPrincipals(set, idp.getUser(userId).getDeclaredGroups(), syncConfig.user().getMembershipNestingDepth());