diff --git a/alfa-service/src/main/java/de/ozgcloud/alfa/bescheid/BescheidVorgangProcessor.java b/alfa-service/src/main/java/de/ozgcloud/alfa/bescheid/BescheidVorgangProcessor.java index da13ce1334dfdc76f838a3778f62f021b2237fbe..1977b5ffb63dbf495578eeac0ebce27d6d7a4af5 100644 --- a/alfa-service/src/main/java/de/ozgcloud/alfa/bescheid/BescheidVorgangProcessor.java +++ b/alfa-service/src/main/java/de/ozgcloud/alfa/bescheid/BescheidVorgangProcessor.java @@ -16,6 +16,7 @@ import de.ozgcloud.alfa.common.FeatureToggleProperties; import de.ozgcloud.alfa.common.ModelBuilder; import de.ozgcloud.alfa.common.command.CommandController.CommandByRelationController; import de.ozgcloud.alfa.common.user.CurrentUserService; +import de.ozgcloud.alfa.common.user.UserRole; import de.ozgcloud.alfa.vorgang.Vorgang; import de.ozgcloud.alfa.vorgang.VorgangWithEingang; import lombok.RequiredArgsConstructor; @@ -35,7 +36,7 @@ class BescheidVorgangProcessor implements RepresentationModelProcessor<EntityMod @Override public EntityModel<VorgangWithEingang> process(EntityModel<VorgangWithEingang> model) { var vorgang = model.getContent(); - if (Objects.isNull(vorgang) || currentUserService.isCurrentUserEinheitlicherAnsprechpartner()) { + if (Objects.isNull(vorgang) || currentUserService.hasRole(UserRole.EINHEITLICHER_ANSPRECHPARTNER)) { return model; } return ModelBuilder.fromModel(model) diff --git a/alfa-service/src/main/java/de/ozgcloud/alfa/common/user/CurrentUserService.java b/alfa-service/src/main/java/de/ozgcloud/alfa/common/user/CurrentUserService.java index 3f25811408c0e6a030d04b7b7bf7018ce9b0a068..f4a517c4052a3658f8ecfc3d81e5ac5a6593c849 100644 --- a/alfa-service/src/main/java/de/ozgcloud/alfa/common/user/CurrentUserService.java +++ b/alfa-service/src/main/java/de/ozgcloud/alfa/common/user/CurrentUserService.java @@ -60,10 +60,6 @@ public class CurrentUserService { return CurrentUserHelper.hasRole(role) || hasRoleReachable(role); } - public boolean isCurrentUserEinheitlicherAnsprechpartner() { - return hasRole(UserRole.EINHEITLICHER_ANSPRECHPARTNER); - } - private boolean hasRoleReachable(String role) { var reachableRoles = roleHierarchy.getReachableGrantedAuthorities(getAuthorities()); diff --git a/alfa-service/src/test/java/de/ozgcloud/alfa/bescheid/BescheidVorgangProcessorTest.java b/alfa-service/src/test/java/de/ozgcloud/alfa/bescheid/BescheidVorgangProcessorTest.java index 3f2e3ba33e4572ca299227654e6bc793f689e9ef..ec24740d58cd87f75c86a04801232b5076fe03f3 100644 --- a/alfa-service/src/test/java/de/ozgcloud/alfa/bescheid/BescheidVorgangProcessorTest.java +++ b/alfa-service/src/test/java/de/ozgcloud/alfa/bescheid/BescheidVorgangProcessorTest.java @@ -23,6 +23,7 @@ import org.springframework.hateoas.Link; import de.ozgcloud.alfa.common.FeatureToggleProperties; import de.ozgcloud.alfa.common.UserProfileUrlProvider; import de.ozgcloud.alfa.common.user.CurrentUserService; +import de.ozgcloud.alfa.common.user.UserRole; import de.ozgcloud.alfa.vorgang.Vorgang; import de.ozgcloud.alfa.vorgang.VorgangHeaderTestFactory; import de.ozgcloud.alfa.vorgang.VorgangWithEingang; @@ -60,7 +61,7 @@ class BescheidVorgangProcessorTest { @Test void shouldReturnTheSameModelWhenUserIsEinheitlicherAnsprechpartner() { var inputModel = EntityModel.of(VorgangWithEingangTestFactory.create()); - when(currentUserService.isCurrentUserEinheitlicherAnsprechpartner()).thenReturn(true); + when(currentUserService.hasRole(UserRole.EINHEITLICHER_ANSPRECHPARTNER)).thenReturn(true); var processedModel = processor.process(inputModel); diff --git a/alfa-service/src/test/java/de/ozgcloud/alfa/common/user/CurrentUserServiceTest.java b/alfa-service/src/test/java/de/ozgcloud/alfa/common/user/CurrentUserServiceTest.java index 57866373540cc62930246fd09c489f3a3f97f19a..e363dc185c06592f6e7e11e35f25c131bd6a83d6 100644 --- a/alfa-service/src/test/java/de/ozgcloud/alfa/common/user/CurrentUserServiceTest.java +++ b/alfa-service/src/test/java/de/ozgcloud/alfa/common/user/CurrentUserServiceTest.java @@ -139,35 +139,4 @@ class CurrentUserServiceTest { } } } - - @Nested - class TestICurrentUserEinheitlicherAnsprechpartner { - - @Test - void shouldCheckRole() { - doReturn(false).when(service).hasRole(any()); - - service.isCurrentUserEinheitlicherAnsprechpartner(); - - verify(service).hasRole(UserRole.EINHEITLICHER_ANSPRECHPARTNER); - } - - @Test - void shouldReturnTrue() { - doReturn(true).when(service).hasRole(UserRole.EINHEITLICHER_ANSPRECHPARTNER); - - var isEinhetlicherAnsprechpartner = service.isCurrentUserEinheitlicherAnsprechpartner(); - - assertThat(isEinhetlicherAnsprechpartner).isTrue(); - } - - @Test - void shouldReturnFalse() { - doReturn(false).when(service).hasRole(any()); - - var isEinhetlicherAnsprechpartner = service.isCurrentUserEinheitlicherAnsprechpartner(); - - assertThat(isEinhetlicherAnsprechpartner).isFalse(); - } - } } \ No newline at end of file