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 e4f61633b748919139154bf8e4008171129c10d8..455d4028812ce220daae65e2865c819a2a26170e 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 cab1ec99ea6c1110930b2e78543c201a62fc5403..627383a227ebb654edcd5adcbf81f939cb51f98a 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); }