diff --git a/nachrichten-manager-postfach-interface/src/main/java/de/ozgcloud/nachrichten/postfach/PostfachAddress.java b/nachrichten-manager-postfach-interface/src/main/java/de/ozgcloud/nachrichten/postfach/PostfachAddress.java index 86ee9d4898f60ea2c3a6ef2bdeeb57506a9a07d0..e9831aa79397ce0ec67c0207b0640d9eee159343 100644 --- a/nachrichten-manager-postfach-interface/src/main/java/de/ozgcloud/nachrichten/postfach/PostfachAddress.java +++ b/nachrichten-manager-postfach-interface/src/main/java/de/ozgcloud/nachrichten/postfach/PostfachAddress.java @@ -23,8 +23,8 @@ */ package de.ozgcloud.nachrichten.postfach; +import jakarta.validation.Valid; import jakarta.validation.constraints.NotBlank; -import jakarta.validation.constraints.NotNull; import lombok.Builder; import lombok.Getter; @@ -43,7 +43,7 @@ public class PostfachAddress { private String version; private int type; - @NotNull + @Valid private PostfachAddressIdentifier identifier; @NotBlank private String serviceKontoType; diff --git a/nachrichten-manager-server/src/main/java/de/ozgcloud/nachrichten/postfach/StringBasedIdentifier.java b/nachrichten-manager-server/src/main/java/de/ozgcloud/nachrichten/postfach/StringBasedIdentifier.java index 5be71d52deac519a2ae87ec1a95772dc214c7325..5724f8624648d2b5deb69a838aeefc51b8123bc9 100644 --- a/nachrichten-manager-server/src/main/java/de/ozgcloud/nachrichten/postfach/StringBasedIdentifier.java +++ b/nachrichten-manager-server/src/main/java/de/ozgcloud/nachrichten/postfach/StringBasedIdentifier.java @@ -23,6 +23,8 @@ */ package de.ozgcloud.nachrichten.postfach; +import jakarta.validation.constraints.NotBlank; + import lombok.Builder; import lombok.Getter; @@ -30,6 +32,7 @@ import lombok.Getter; @Getter public class StringBasedIdentifier implements PostfachAddressIdentifier { + @NotBlank private String postfachId; @Override 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 87daf339100ab4960c2bf96c7180bdf907765fc8..55f8368dacd8eae7220957419bee650dec58fb33 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 @@ -41,6 +41,7 @@ import de.ozgcloud.nachrichten.postfach.PostfachException; import de.ozgcloud.nachrichten.postfach.PostfachMessageCode; import de.ozgcloud.nachrichten.postfach.PostfachNachricht; import de.ozgcloud.nachrichten.postfach.PostfachNachrichtTestFactory; +import de.ozgcloud.nachrichten.postfach.StringBasedIdentifierTestFactory; import de.ozgcloud.nachrichten.postfach.osi.MessageTestFactory; class InfoManagerServiceTest { @@ -134,7 +135,7 @@ class InfoManagerServiceTest { var id = service.getPostfachId(address); - assertThat(id).isEqualTo(PostfachAddressTestFactory.STRING_BASED_IDENTIFIER_POSTFACH_ID_VALUE); + assertThat(id).isEqualTo(StringBasedIdentifierTestFactory.STRING_BASED_IDENTIFIER_POSTFACH_ID_VALUE); } @Test diff --git a/nachrichten-manager-server/src/test/java/de/ozgcloud/nachrichten/postfach/PersistPostfachNachrichtServiceImplTest.java b/nachrichten-manager-server/src/test/java/de/ozgcloud/nachrichten/postfach/PersistPostfachNachrichtServiceImplTest.java index d80fe364dae457b76bf9ce0fa82460f0d7076fe8..9a652531f5953acf294461fd3051296dac655826 100644 --- a/nachrichten-manager-server/src/test/java/de/ozgcloud/nachrichten/postfach/PersistPostfachNachrichtServiceImplTest.java +++ b/nachrichten-manager-server/src/test/java/de/ozgcloud/nachrichten/postfach/PersistPostfachNachrichtServiceImplTest.java @@ -467,7 +467,7 @@ class PersistPostfachNachrichtServiceImplTest { void shouldCallVorgangRemoteService() { doFindAnswers().toList(); - verify(vorgangService).findVorgangIds(serviceKontoType, PostfachAddressTestFactory.STRING_BASED_IDENTIFIER_POSTFACH_ID_VALUE); + verify(vorgangService).findVorgangIds(serviceKontoType, StringBasedIdentifierTestFactory.STRING_BASED_IDENTIFIER_POSTFACH_ID_VALUE); } @Test @@ -485,7 +485,7 @@ class PersistPostfachNachrichtServiceImplTest { } private Stream<PostfachNachricht> doFindAnswers() { - return service.findAnswers(serviceKontoType, PostfachAddressTestFactory.STRING_BASED_IDENTIFIER_POSTFACH_ID_VALUE, + return service.findAnswers(serviceKontoType, StringBasedIdentifierTestFactory.STRING_BASED_IDENTIFIER_POSTFACH_ID_VALUE, GrpcPostfachMailTestFactory.REFERENCED_NACHRICHT_ID); } } diff --git a/nachrichten-manager-server/src/test/java/de/ozgcloud/nachrichten/postfach/PostfachAddressTestFactory.java b/nachrichten-manager-server/src/test/java/de/ozgcloud/nachrichten/postfach/PostfachAddressTestFactory.java index 572b41d8e42e75401ac97d5b61c31b259cdd4b45..e3458bb83e5a4a48d7dc3e264d414c0fe084471b 100644 --- a/nachrichten-manager-server/src/test/java/de/ozgcloud/nachrichten/postfach/PostfachAddressTestFactory.java +++ b/nachrichten-manager-server/src/test/java/de/ozgcloud/nachrichten/postfach/PostfachAddressTestFactory.java @@ -23,14 +23,10 @@ */ package de.ozgcloud.nachrichten.postfach; -import de.ozgcloud.nachrichten.postfach.osi.MessageTestFactory; - public class PostfachAddressTestFactory { public final static int TYPE = 1; - public static final String STRING_BASED_IDENTIFIER_POSTFACH_ID_VALUE = MessageTestFactory.POSTFACH_ID; - public final static StringBasedIdentifier IDENTIFIER = StringBasedIdentifier.builder().postfachId(STRING_BASED_IDENTIFIER_POSTFACH_ID_VALUE) - .build(); + public final static StringBasedIdentifier IDENTIFIER = StringBasedIdentifierTestFactory.create(); public static final String VERSION = "1.0"; public static PostfachAddress create() { diff --git a/nachrichten-manager-server/src/test/java/de/ozgcloud/nachrichten/postfach/PostfachEventListenerTest.java b/nachrichten-manager-server/src/test/java/de/ozgcloud/nachrichten/postfach/PostfachEventListenerTest.java index 46a2e817bef81fad7213bc3ced02bba1ccac22b0..b38987c546ab6e67ccac383ad59e292decdcd379 100644 --- a/nachrichten-manager-server/src/test/java/de/ozgcloud/nachrichten/postfach/PostfachEventListenerTest.java +++ b/nachrichten-manager-server/src/test/java/de/ozgcloud/nachrichten/postfach/PostfachEventListenerTest.java @@ -146,7 +146,7 @@ class PostfachEventListenerTest { PostfachAddress.TYPE_FIELD, PostfachAddressTestFactory.TYPE, PostfachAddress.SERVICEKONTO_TYPE_FIELD, PostfachTestFactory.POSTFACH_TYPE, PostfachAddress.IDENTIFIER_FIELD, // - Map.of(PostfachAddress.FIELD_POSTFACH_ID, PostfachAddressTestFactory.STRING_BASED_IDENTIFIER_POSTFACH_ID_VALUE)), + Map.of(PostfachAddress.FIELD_POSTFACH_ID, StringBasedIdentifierTestFactory.STRING_BASED_IDENTIFIER_POSTFACH_ID_VALUE)), PostfachNachricht.FIELD_SUBJECT, MessageTestFactory.SUBJECT, PostfachNachricht.FIELD_MAIL_BODY, PostfachNachrichtTestFactory.MAIL_BODY, PostfachNachricht.FIELD_REPLY_OPTION, PostfachNachrichtTestFactory.REPLY_OPTION.name(), diff --git a/nachrichten-manager-server/src/test/java/de/ozgcloud/nachrichten/postfach/PostfachNachrichtTestFactory.java b/nachrichten-manager-server/src/test/java/de/ozgcloud/nachrichten/postfach/PostfachNachrichtTestFactory.java index aec4cf7b58accc613a8164dc55c92c9803c6be5b..6c4e1e5fc114db3bbab34738d4d4ad6ee1106dad 100644 --- a/nachrichten-manager-server/src/test/java/de/ozgcloud/nachrichten/postfach/PostfachNachrichtTestFactory.java +++ b/nachrichten-manager-server/src/test/java/de/ozgcloud/nachrichten/postfach/PostfachNachrichtTestFactory.java @@ -104,6 +104,6 @@ public class PostfachNachrichtTestFactory { } private static Map<String, Object> getPostfachAddressIdentifierAsMap() { - return Map.of(PostfachAddress.FIELD_POSTFACH_ID, PostfachAddressTestFactory.STRING_BASED_IDENTIFIER_POSTFACH_ID_VALUE); + return Map.of(PostfachAddress.FIELD_POSTFACH_ID, StringBasedIdentifierTestFactory.STRING_BASED_IDENTIFIER_POSTFACH_ID_VALUE); } } diff --git a/nachrichten-manager-server/src/test/java/de/ozgcloud/nachrichten/postfach/StringBasedIdentifierTestFactory.java b/nachrichten-manager-server/src/test/java/de/ozgcloud/nachrichten/postfach/StringBasedIdentifierTestFactory.java new file mode 100644 index 0000000000000000000000000000000000000000..7404e001413533580fb2ce6071cf56e1dd68b4b2 --- /dev/null +++ b/nachrichten-manager-server/src/test/java/de/ozgcloud/nachrichten/postfach/StringBasedIdentifierTestFactory.java @@ -0,0 +1,40 @@ +/* + * Copyright (C) 2025 Das Land Schleswig-Holstein vertreten durch den + * Ministerpräsidenten des Landes Schleswig-Holstein + * Staatskanzlei + * Abteilung Digitalisierung und zentrales IT-Management der Landesregierung + * + * Lizenziert unter der EUPL, Version 1.2 oder - sobald + * diese von der Europäischen Kommission genehmigt wurden - + * Folgeversionen der EUPL ("Lizenz"); + * Sie dürfen dieses Werk ausschließlich gemäß + * dieser Lizenz nutzen. + * Eine Kopie der Lizenz finden Sie hier: + * + * https://joinup.ec.europa.eu/collection/eupl/eupl-text-eupl-12 + * + * Sofern nicht durch anwendbare Rechtsvorschriften + * gefordert oder in schriftlicher Form vereinbart, wird + * die unter der Lizenz verbreitete Software "so wie sie + * ist", OHNE JEGLICHE GEWÄHRLEISTUNG ODER BEDINGUNGEN - + * ausdrücklich oder stillschweigend - verbreitet. + * Die sprachspezifischen Genehmigungen und Beschränkungen + * unter der Lizenz sind dem Lizenztext zu entnehmen. + */ +package de.ozgcloud.nachrichten.postfach; + +import de.ozgcloud.nachrichten.postfach.StringBasedIdentifier.StringBasedIdentifierBuilder; +import de.ozgcloud.nachrichten.postfach.osi.MessageTestFactory; + +public class StringBasedIdentifierTestFactory { + + public static final String STRING_BASED_IDENTIFIER_POSTFACH_ID_VALUE = MessageTestFactory.POSTFACH_ID; + + public static StringBasedIdentifier create() { + return createBuilder().build(); + } + + public static StringBasedIdentifierBuilder createBuilder() { + return StringBasedIdentifier.builder().postfachId(STRING_BASED_IDENTIFIER_POSTFACH_ID_VALUE); + } +} diff --git a/nachrichten-manager-server/src/test/java/de/ozgcloud/nachrichten/postfach/bayernid/BayernIdPostfachRemoteServiceITCase.java b/nachrichten-manager-server/src/test/java/de/ozgcloud/nachrichten/postfach/bayernid/BayernIdPostfachRemoteServiceITCase.java index 86f9e13823027aa73d2313fdf4dbeb0232d66953..6b64dbce6cfee0f3b5624eaeb2a74d839b4006e2 100644 --- a/nachrichten-manager-server/src/test/java/de/ozgcloud/nachrichten/postfach/bayernid/BayernIdPostfachRemoteServiceITCase.java +++ b/nachrichten-manager-server/src/test/java/de/ozgcloud/nachrichten/postfach/bayernid/BayernIdPostfachRemoteServiceITCase.java @@ -28,7 +28,6 @@ import static org.junit.jupiter.api.Assertions.*; import jakarta.validation.ConstraintViolationException; import org.junit.jupiter.api.Nested; -import org.junit.jupiter.api.Test; import org.junit.jupiter.params.ParameterizedTest; import org.junit.jupiter.params.provider.NullAndEmptySource; import org.springframework.beans.factory.annotation.Autowired; @@ -39,6 +38,7 @@ import de.ozgcloud.common.test.ITCase; import de.ozgcloud.nachrichten.email.EmailGrpcService; import de.ozgcloud.nachrichten.postfach.PostfachAddressTestFactory; import de.ozgcloud.nachrichten.postfach.PostfachNachrichtTestFactory; +import de.ozgcloud.nachrichten.postfach.StringBasedIdentifierTestFactory; @ITCase @SpringBootTest(properties = { "ozgcloud.bayernid.enabled = true" }) @@ -65,10 +65,14 @@ class BayernIdPostfachRemoteServiceITCase { assertThrows(ConstraintViolationException.class, () -> service.sendMessage(nachricht)); } - @Test - void shouldThrowConstraintViolationExceptionOnMissingIdentifier() { + @ParameterizedTest + @NullAndEmptySource + void shouldThrowConstraintViolationExceptionOnMissingIdentifier(String postfachId) { + var postfachAddress = PostfachAddressTestFactory.createBuilder() + .identifier(StringBasedIdentifierTestFactory.createBuilder().postfachId(postfachId).build()) + .build(); var nachricht = PostfachNachrichtTestFactory.createBuilder() - .postfachAddress(PostfachAddressTestFactory.createBuilder().identifier(null).build()) + .postfachAddress(postfachAddress) .build(); assertThrows(ConstraintViolationException.class, () -> service.sendMessage(nachricht)); diff --git a/nachrichten-manager-server/src/test/java/de/ozgcloud/nachrichten/postfach/osi/OsiPostfachMessageMapperTest.java b/nachrichten-manager-server/src/test/java/de/ozgcloud/nachrichten/postfach/osi/OsiPostfachMessageMapperTest.java index 29255fb6e8902a95f18d0a6fae97db6a89ffbcf4..48ff7d0f2654c5793b3823df1e8ca4a6c21d8bd6 100644 --- a/nachrichten-manager-server/src/test/java/de/ozgcloud/nachrichten/postfach/osi/OsiPostfachMessageMapperTest.java +++ b/nachrichten-manager-server/src/test/java/de/ozgcloud/nachrichten/postfach/osi/OsiPostfachMessageMapperTest.java @@ -42,6 +42,7 @@ import de.ozgcloud.nachrichten.postfach.PostfachAddressTestFactory; import de.ozgcloud.nachrichten.postfach.PostfachNachricht; import de.ozgcloud.nachrichten.postfach.PostfachNachricht.Direction; import de.ozgcloud.nachrichten.postfach.PostfachNachrichtTestFactory; +import de.ozgcloud.nachrichten.postfach.StringBasedIdentifierTestFactory; import de.ozgcloud.vorgang.common.grpc.GrpcObjectMapper; class OsiPostfachMessageMapperTest { @@ -163,7 +164,7 @@ class OsiPostfachMessageMapperTest { void shouldMapPostfachAddressToPostfachId() { var message = toOsiMessage(PostfachNachrichtTestFactory.create()); - assertThat(message.getPostfachId()).isEqualTo(PostfachAddressTestFactory.STRING_BASED_IDENTIFIER_POSTFACH_ID_VALUE); + assertThat(message.getPostfachId()).isEqualTo(StringBasedIdentifierTestFactory.STRING_BASED_IDENTIFIER_POSTFACH_ID_VALUE); } } diff --git a/nachrichten-manager-server/src/test/java/de/ozgcloud/nachrichten/postfach/osi/OsiPostfachRemoteServiceITCase.java b/nachrichten-manager-server/src/test/java/de/ozgcloud/nachrichten/postfach/osi/OsiPostfachRemoteServiceITCase.java index ee9c037f8023b9009e170edb268ceb27642d5af6..2ddd91cc5646af5e2d48403189b638a59701b134 100644 --- a/nachrichten-manager-server/src/test/java/de/ozgcloud/nachrichten/postfach/osi/OsiPostfachRemoteServiceITCase.java +++ b/nachrichten-manager-server/src/test/java/de/ozgcloud/nachrichten/postfach/osi/OsiPostfachRemoteServiceITCase.java @@ -28,7 +28,6 @@ import static org.junit.jupiter.api.Assertions.*; import jakarta.validation.ConstraintViolationException; import org.junit.jupiter.api.Nested; -import org.junit.jupiter.api.Test; import org.junit.jupiter.params.ParameterizedTest; import org.junit.jupiter.params.provider.NullAndEmptySource; import org.springframework.beans.factory.annotation.Autowired; @@ -40,6 +39,7 @@ import de.ozgcloud.common.test.ITCase; import de.ozgcloud.nachrichten.email.EmailGrpcService; import de.ozgcloud.nachrichten.postfach.PostfachAddressTestFactory; import de.ozgcloud.nachrichten.postfach.PostfachNachrichtTestFactory; +import de.ozgcloud.nachrichten.postfach.StringBasedIdentifierTestFactory; @ITCase @SpringBootTest(properties = { @@ -72,10 +72,14 @@ class OsiPostfachRemoteServiceITCase { assertThrows(ConstraintViolationException.class, () -> service.sendMessage(nachricht)); } - @Test - void shouldThrowConstraintViolationExceptionOnMissingIdentifier() { + @ParameterizedTest + @NullAndEmptySource + void shouldThrowConstraintViolationExceptionOnMissingIdentifier(String postfachId) { + var postfachAddress = PostfachAddressTestFactory.createBuilder() + .identifier(StringBasedIdentifierTestFactory.createBuilder().postfachId(postfachId).build()) + .build(); var nachricht = PostfachNachrichtTestFactory.createBuilder() - .postfachAddress(PostfachAddressTestFactory.createBuilder().identifier(null).build()) + .postfachAddress(postfachAddress) .build(); assertThrows(ConstraintViolationException.class, () -> service.sendMessage(nachricht));