From db20d3d01c4e721d71123a6980019fec3418f53b Mon Sep 17 00:00:00 2001
From: Felix Reichenbach <felix.reichenbach@mgm-tp.com>
Date: Fri, 3 Jan 2025 15:12:49 +0100
Subject: [PATCH] OZG-7136 remove workaround implemented in OZG-7126
---
.../nachrichten/postfach/PostfachService.java | 11 +++--------
.../nachrichten/postfach/PostfachServiceTest.java | 14 +-------------
2 files changed, 4 insertions(+), 21 deletions(-)
diff --git a/nachrichten-manager-server/src/main/java/de/ozgcloud/nachrichten/postfach/PostfachService.java b/nachrichten-manager-server/src/main/java/de/ozgcloud/nachrichten/postfach/PostfachService.java
index e4f6163..455d402 100644
--- a/nachrichten-manager-server/src/main/java/de/ozgcloud/nachrichten/postfach/PostfachService.java
+++ b/nachrichten-manager-server/src/main/java/de/ozgcloud/nachrichten/postfach/PostfachService.java
@@ -276,20 +276,15 @@ class PostfachService {
}
Optional<PostfachRemoteService> findPostfachRemoteService(String postfachType) {
- var remoteServices = getPostfachRemoteServices().toList();
- if (Objects.isNull(postfachType) && remoteServices.size() != 1) {
- throw new TechnicalException("PostfachType is null and no distinct PostfachService is configured.");
+ if (Objects.isNull(postfachType)) {
+ throw new TechnicalException("PostfachType is null.");
}
- return remoteServices.stream()
+ return getPostfachRemoteServices()
.filter(remoteService -> hasPostfachType(remoteService, postfachType))
.findFirst();
}
private boolean hasPostfachType(PostfachRemoteService postfachRemoteService, String postfachType) {
- if (Objects.isNull(postfachType)) {
- LOG.error("PostfachType is null - use existing PostfachService with type %s.".formatted(postfachRemoteService.getPostfachType()));
- return true;
- }
return StringUtils.equals(postfachRemoteService.getPostfachType(), postfachType);
}
diff --git a/nachrichten-manager-server/src/test/java/de/ozgcloud/nachrichten/postfach/PostfachServiceTest.java b/nachrichten-manager-server/src/test/java/de/ozgcloud/nachrichten/postfach/PostfachServiceTest.java
index cab1ec9..627383a 100644
--- a/nachrichten-manager-server/src/test/java/de/ozgcloud/nachrichten/postfach/PostfachServiceTest.java
+++ b/nachrichten-manager-server/src/test/java/de/ozgcloud/nachrichten/postfach/PostfachServiceTest.java
@@ -884,19 +884,7 @@ class PostfachServiceTest {
class TestOnPostfachTypeNull {
@Test
- void shouldReturnRemoteService() {
- when(remoteService.getPostfachType()).thenReturn("BAYERN_ID");
- doReturn(Stream.of(remoteService)).when(service).getPostfachRemoteServices();
-
- var foundRemoteService = service.findPostfachRemoteService(null);
-
- assertThat(foundRemoteService).hasValue(remoteService);
- }
-
- @Test
- void shouldThrowExceptionMultipleConfiguredRemoteServices() {
- doReturn(Stream.of(anotherRemoteService, remoteService)).when(service).getPostfachRemoteServices();
-
+ void shouldThrowException() {
assertThatThrownBy(() -> service.findPostfachRemoteService(null))
.isInstanceOf(TechnicalException.class);
}
--
GitLab