From ef68e4f06c9ca4593f7dfe300153d0eea3b0c8f3 Mon Sep 17 00:00:00 2001
From: OZGCloud <ozgcloud@mgm-tp.com>
Date: Thu, 30 May 2024 16:19:30 +0200
Subject: [PATCH] OZG-5294: remove method
---
.../bescheid/BescheidVorgangProcessor.java | 3 +-
.../alfa/common/user/CurrentUserService.java | 4 ---
.../BescheidVorgangProcessorTest.java | 3 +-
.../common/user/CurrentUserServiceTest.java | 31 -------------------
4 files changed, 4 insertions(+), 37 deletions(-)
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 da13ce1334..1977b5ffb6 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 3f25811408..f4a517c405 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 3f2e3ba33e..ec24740d58 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 5786637354..e363dc185c 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
--
GitLab