diff --git a/nachrichten-manager-server/src/main/java/de/ozgcloud/nachrichten/postfach/PostfachNachrichtMapper.java b/nachrichten-manager-server/src/main/java/de/ozgcloud/nachrichten/postfach/PostfachNachrichtMapper.java index 6ef2b4bf659ff1d7dfb079269f68cd9b7919b12b..1fff1bd82ecef30baea5941bd2be761ab05d8944 100644 --- a/nachrichten-manager-server/src/main/java/de/ozgcloud/nachrichten/postfach/PostfachNachrichtMapper.java +++ b/nachrichten-manager-server/src/main/java/de/ozgcloud/nachrichten/postfach/PostfachNachrichtMapper.java @@ -240,6 +240,9 @@ public abstract class PostfachNachrichtMapper { private Map<String, Object> buildPostfachAddressIdentifierMap(PostfachAddress postfachAddress) { var identifier = postfachAddress.getIdentifier(); - return Map.of(PostfachAddress.FIELD_POSTFACH_ID, identifier.isStringBasedIdentifier() ? identifier.toString() : ""); + if (!identifier.isStringBasedIdentifier()) { + throw new IllegalStateException("Illegal Type of Postfach Identifier: " + identifier.getClass().getName()); + } + return Map.of(PostfachAddress.FIELD_POSTFACH_ID, identifier.toString()); } } \ No newline at end of file diff --git a/nachrichten-manager-server/src/test/java/de/ozgcloud/nachrichten/postfach/PostfachNachrichtMapperTest.java b/nachrichten-manager-server/src/test/java/de/ozgcloud/nachrichten/postfach/PostfachNachrichtMapperTest.java index 469309f6979305ca463a4b5d0744e5bba8f6cb49..30a5d0bf35e31b3e9c21ab2c068fe45cff051abe 100644 --- a/nachrichten-manager-server/src/test/java/de/ozgcloud/nachrichten/postfach/PostfachNachrichtMapperTest.java +++ b/nachrichten-manager-server/src/test/java/de/ozgcloud/nachrichten/postfach/PostfachNachrichtMapperTest.java @@ -339,10 +339,9 @@ class PostfachNachrichtMapperTest { assertDoesNotThrow(() -> mapper.mapToMap(postfachMail)); } - @DisplayName("should map non string based identifier to empty") + @DisplayName("should throw if not string-based identifier") @Test - - void shouldMapNonStringBasedIdentifierToEmpty() { + void shouldThrowIfNotStringBasedIdentifier() { var postfachMail = PostfachNachrichtTestFactory.createBuilder() .postfachAddress(PostfachAddressTestFactory.createBuilder() .identifier(new PostfachAddressIdentifier() { @@ -350,9 +349,9 @@ class PostfachNachrichtMapperTest { .build() ).build(); - var postfachId = mapPostfachId(postfachMail); + assertThatThrownBy(() -> mapPostfachId(postfachMail)) + .isInstanceOf(IllegalStateException.class); - assertThat(postfachId).isEmpty(); } @DisplayName("should map string based identifier")