diff --git a/CHANGELOG.md b/CHANGELOG.md index 4f2c217e1..2ad68e811 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -1,5 +1,8 @@ # Changelog All notable changes to this project will be documented in this file. +## [10.6.4] +### Changed +- Fix parameters order in remove permissions in AccessGroupService in other calling methods and junits ## [10.6.3] ### Changed diff --git a/stream-access-control/access-control-core/src/main/java/com/backbase/stream/service/AccessGroupService.java b/stream-access-control/access-control-core/src/main/java/com/backbase/stream/service/AccessGroupService.java index f61e507ca..c431c3010 100644 --- a/stream-access-control/access-control-core/src/main/java/com/backbase/stream/service/AccessGroupService.java +++ b/stream-access-control/access-control-core/src/main/java/com/backbase/stream/service/AccessGroupService.java @@ -560,7 +560,7 @@ public Mono assignPermissions(StreamTask streamTask, ServiceAgre String serviceAgreementId = serviceAgreement.getInternalId(); String userId = jobProfileUser.getUser().getInternalId(); - return assignPermissionsServiceApi.putUserPermissions(serviceAgreementId, userId, functionGroups) + return assignPermissionsServiceApi.putUserPermissions(userId, serviceAgreementId, functionGroups) .onErrorResume(WebClientResponseException.class, e -> { streamTask.error(ACCESS_GROUP, "assign-permissions", "failed", serviceAgreement.getExternalId(), serviceAgreementId, e, e.getResponseBodyAsString(), "Failed to to assign permissions"); diff --git a/stream-access-control/access-control-core/src/test/java/com/backbase/stream/service/AccessGroupServiceTest.java b/stream-access-control/access-control-core/src/test/java/com/backbase/stream/service/AccessGroupServiceTest.java index 4bef4289a..107ec9548 100644 --- a/stream-access-control/access-control-core/src/test/java/com/backbase/stream/service/AccessGroupServiceTest.java +++ b/stream-access-control/access-control-core/src/test/java/com/backbase/stream/service/AccessGroupServiceTest.java @@ -2107,8 +2107,8 @@ void shouldCreateDataGroup() { @Test void shouldRemoveUserPermissions() { when(assignPermissionsServiceApi.putUserPermissions(any(), any(), any())).thenReturn(Mono.empty()); - subject.removePermissionsForUser("sa-id", "user-id").block(); - verify(assignPermissionsServiceApi).putUserPermissions("sa-id", "user-id", List.of()); + subject.removePermissionsForUser("user-id","sa-id").block(); + verify(assignPermissionsServiceApi).putUserPermissions("user-id","sa-id",List.of()); } @Test