diff --git a/nachrichten-manager-postfach-interface/src/main/java/de/ozgcloud/nachrichten/postfach/PostfachAddressIdentifier.java b/nachrichten-manager-postfach-interface/src/main/java/de/ozgcloud/nachrichten/postfach/PostfachAddressIdentifier.java index 2ead26b3a06676ddcbc149aae1b9c717cfd7f0e3..e163cbd84939285a66bd8aaefa4674529f1ca876 100644 --- a/nachrichten-manager-postfach-interface/src/main/java/de/ozgcloud/nachrichten/postfach/PostfachAddressIdentifier.java +++ b/nachrichten-manager-postfach-interface/src/main/java/de/ozgcloud/nachrichten/postfach/PostfachAddressIdentifier.java @@ -25,9 +25,5 @@ package de.ozgcloud.nachrichten.postfach; public interface PostfachAddressIdentifier { - default boolean isStringBasedIdentifier() { - return false; - } - String getStringRepresentation(); } diff --git a/nachrichten-manager-postfach-interface/src/main/java/de/ozgcloud/nachrichten/postfach/StringBasedIdentifier.java b/nachrichten-manager-postfach-interface/src/main/java/de/ozgcloud/nachrichten/postfach/StringBasedIdentifier.java index 5940c9ff7a478504470449e3df06beb87f44ddd5..5a11033b7a4b4d539b48a711f887d6170328091f 100644 --- a/nachrichten-manager-postfach-interface/src/main/java/de/ozgcloud/nachrichten/postfach/StringBasedIdentifier.java +++ b/nachrichten-manager-postfach-interface/src/main/java/de/ozgcloud/nachrichten/postfach/StringBasedIdentifier.java @@ -33,11 +33,6 @@ public class StringBasedIdentifier implements PostfachAddressIdentifier { @NotBlank private String postfachId; - @Override - public boolean isStringBasedIdentifier() { - return true; - } - @Override public String toString() { return postfachId; diff --git a/nachrichten-manager-server/src/main/java/de/ozgcloud/nachrichten/info/InfoManagerService.java b/nachrichten-manager-server/src/main/java/de/ozgcloud/nachrichten/info/InfoManagerService.java index 351908590f952b45b78bcf3a0911709e97d7551f..a375ec671fd70e5c5b5661cfb0dd57dcb40551d4 100644 --- a/nachrichten-manager-server/src/main/java/de/ozgcloud/nachrichten/info/InfoManagerService.java +++ b/nachrichten-manager-server/src/main/java/de/ozgcloud/nachrichten/info/InfoManagerService.java @@ -70,12 +70,6 @@ public class InfoManagerService { } String getPostfachId(@NonNull PostfachAddress address) { - var postfachIdentifier = address.getIdentifier(); - - if (postfachIdentifier.isStringBasedIdentifier()) { - return postfachIdentifier.getStringRepresentation(); - } else { - throw new IllegalStateException("Illegal Type of Postfach Identifier: " + postfachIdentifier.getClass().getName()); - } + return address.getIdentifier().getStringRepresentation(); } } diff --git a/nachrichten-manager-server/src/test/java/de/ozgcloud/nachrichten/info/InfoManagerServiceTest.java b/nachrichten-manager-server/src/test/java/de/ozgcloud/nachrichten/info/InfoManagerServiceTest.java index ccf343a7578c1dfd68f88a8a895395c25b1405c3..2d36d96e2638431d57ecf22fed504b78e4975773 100644 --- a/nachrichten-manager-server/src/test/java/de/ozgcloud/nachrichten/info/InfoManagerServiceTest.java +++ b/nachrichten-manager-server/src/test/java/de/ozgcloud/nachrichten/info/InfoManagerServiceTest.java @@ -28,7 +28,6 @@ import static org.junit.jupiter.api.Assertions.*; import static org.mockito.ArgumentMatchers.*; import static org.mockito.Mockito.*; -import org.apache.commons.lang3.StringUtils; import org.junit.jupiter.api.Nested; import org.junit.jupiter.api.Test; import org.mockito.InjectMocks; @@ -36,7 +35,6 @@ import org.mockito.Mock; import org.mockito.Spy; import de.ozgcloud.nachrichten.NachrichtenManagerProperties; -import de.ozgcloud.nachrichten.postfach.PostfachAddressIdentifier; import de.ozgcloud.nachrichten.postfach.PostfachAddressTestFactory; import de.ozgcloud.nachrichten.postfach.PostfachException; import de.ozgcloud.nachrichten.postfach.PostfachMessageCode; @@ -139,19 +137,5 @@ class InfoManagerServiceTest { assertThat(id).isEqualTo(StringBasedIdentifierTestFactory.STRING_BASED_IDENTIFIER_POSTFACH_ID_VALUE); } - @Test - void shouldThrowExceptionForUnkownIdentifer() { - var identifier = new PostfachAddressIdentifier() { - - @Override - public String getStringRepresentation() { - return StringUtils.EMPTY; - } - }; - - var address = PostfachAddressTestFactory.createBuilder().identifier(identifier).build(); - - assertThatThrownBy(() -> service.getPostfachId(address)).isInstanceOf(IllegalStateException.class); - } } }