From ecb5e37e0732efa08f6acf34f5f9f767fc0280f3 Mon Sep 17 00:00:00 2001
From: Felix Reichenbach <felix.reichenbach@mgm-tp.com>
Date: Tue, 18 Feb 2025 15:47:25 +0100
Subject: [PATCH] OZG-4094 remove isStringBasedIdentifier from
PostfachAddressidentifier
---
.../postfach/PostfachAddressIdentifier.java | 4 ----
.../postfach/StringBasedIdentifier.java | 5 -----
.../nachrichten/info/InfoManagerService.java | 8 +-------
.../nachrichten/info/InfoManagerServiceTest.java | 16 ----------------
4 files changed, 1 insertion(+), 32 deletions(-)
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 2ead26b..e163cbd 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 5940c9f..5a11033 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 3519085..a375ec6 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 ccf343a..2d36d96 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);
- }
}
}
--
GitLab