diff --git a/nachrichten-manager-interface/src/main/protobuf/postfach.model.proto b/nachrichten-manager-interface/src/main/protobuf/postfach.model.proto
index d3cfc107e10928b5897a651ae549a2dbf746072f..b7f2a680d8e3ed5f3a3925efbfe8cfba2255c883 100644
--- a/nachrichten-manager-interface/src/main/protobuf/postfach.model.proto
+++ b/nachrichten-manager-interface/src/main/protobuf/postfach.model.proto
@@ -87,7 +87,6 @@ message GrpcPostfachNachricht {
message GrpcPostfachMail {
string id = 1;
string vorgangId = 2;
- string postfachId = 3 [deprecated = true];
GrpcPostfachAddress postfachAddress = 14;
string createdAt = 4;
string createdBy = 5;
diff --git a/nachrichten-manager-interface/src/test/java/de/ozgcloud/nachrichten/postfach/GrpcPostfachMailTestFactory.java b/nachrichten-manager-interface/src/test/java/de/ozgcloud/nachrichten/postfach/GrpcPostfachMailTestFactory.java
index cb6eccfb3ba9f3e01d5b6917de337b63aaed8ffe..c1cd02759d0ad7ec66bd264b9e1e748c5214fe36 100644
--- a/nachrichten-manager-interface/src/test/java/de/ozgcloud/nachrichten/postfach/GrpcPostfachMailTestFactory.java
+++ b/nachrichten-manager-interface/src/test/java/de/ozgcloud/nachrichten/postfach/GrpcPostfachMailTestFactory.java
@@ -52,7 +52,6 @@ public class GrpcPostfachMailTestFactory {
.setId(ID)
.setReferencedNachricht(REFERENCED_NACHRICHT_ID)
.setVorgangId(VORGANG_ID)
- .setPostfachId(POSTFACH_ID)
.setPostfachAddress(GrpcPostfachAddressTestFactory.create())
.setCreatedAt(CREATED_AT)
.setCreatedBy(CREATED_BY)
diff --git a/nachrichten-manager-postfach-interface/src/main/java/de/ozgcloud/nachrichten/postfach/PostfachNachricht.java b/nachrichten-manager-postfach-interface/src/main/java/de/ozgcloud/nachrichten/postfach/PostfachNachricht.java
index d76a8251fc2c6df30d2f16e9b2e1e77328d2cafa..8117a942608c2eba7ca14f291e284d96900cceef 100644
--- a/nachrichten-manager-postfach-interface/src/main/java/de/ozgcloud/nachrichten/postfach/PostfachNachricht.java
+++ b/nachrichten-manager-postfach-interface/src/main/java/de/ozgcloud/nachrichten/postfach/PostfachNachricht.java
@@ -68,8 +68,6 @@ public class PostfachNachricht {
private String id;
private String vorgangId;
- @Deprecated
- private String postfachId;
private PostfachAddress postfachAddress;
private String messageId;
private String referencedNachricht;
diff --git a/nachrichten-manager-server/src/main/java/de/ozgcloud/nachrichten/antragraum/AntragraumNachrichtMapper.java b/nachrichten-manager-server/src/main/java/de/ozgcloud/nachrichten/antragraum/AntragraumNachrichtMapper.java
index 3c14695971b75f4d1b8543f8f7826a47286a0506..b41a700747966ec81f65ea0dd4de8b8300e647f3 100644
--- a/nachrichten-manager-server/src/main/java/de/ozgcloud/nachrichten/antragraum/AntragraumNachrichtMapper.java
+++ b/nachrichten-manager-server/src/main/java/de/ozgcloud/nachrichten/antragraum/AntragraumNachrichtMapper.java
@@ -66,7 +66,6 @@ interface AntragraumNachrichtMapper {
@Mapping(target = "messageCode", ignore = true)
@Mapping(target = "messageId", source = "rueckfrageId")
@Mapping(target = "postfachAddress", ignore = true)
- @Mapping(target = "postfachId", ignore = true)
@Mapping(target = "replyOption", constant = "FORBIDDEN")
@Mapping(target = "sentSuccessful", constant = "true")
@Mapping(target = "subject", constant = "Antwort") // TODO klären
diff --git a/nachrichten-manager-server/src/main/java/de/ozgcloud/nachrichten/postfach/GrpcPostfachNachrichtMapper.java b/nachrichten-manager-server/src/main/java/de/ozgcloud/nachrichten/postfach/GrpcPostfachNachrichtMapper.java
index 53a1512d66e5c20b10821c3953e9343ccb7ac690..7f4cc6381dc45324294af91163a40e2d5f0d8652 100644
--- a/nachrichten-manager-server/src/main/java/de/ozgcloud/nachrichten/postfach/GrpcPostfachNachrichtMapper.java
+++ b/nachrichten-manager-server/src/main/java/de/ozgcloud/nachrichten/postfach/GrpcPostfachNachrichtMapper.java
@@ -15,7 +15,6 @@ public interface GrpcPostfachNachrichtMapper {
@Mapping(target = "direction", constant = "OUT")
@Mapping(target = "messageCode", ignore = true)
@Mapping(target = "messageId", ignore = true)
- @Mapping(target = "postfachId", ignore = true)
@Mapping(target = "sentAt", ignore = true)
@Mapping(target = "sentSuccessful", ignore = true)
@Mapping(target = "vorgangId", ignore = true)
diff --git a/nachrichten-manager-server/src/main/java/de/ozgcloud/nachrichten/postfach/PostfachEventListener.java b/nachrichten-manager-server/src/main/java/de/ozgcloud/nachrichten/postfach/PostfachEventListener.java
index 9937bbbfbc0ad58912046e60b791c4e8e8af8323..7b4a3be2dbde98a5ed88eaed0fbd429a6ce860be 100644
--- a/nachrichten-manager-server/src/main/java/de/ozgcloud/nachrichten/postfach/PostfachEventListener.java
+++ b/nachrichten-manager-server/src/main/java/de/ozgcloud/nachrichten/postfach/PostfachEventListener.java
@@ -77,7 +77,6 @@ public class PostfachEventListener {
var postfachMailBuilder = PostfachNachricht.builder()
.id(isNewNachricht(command) ? null : command.getRelationId())
.vorgangId(command.getVorgangId())
- .postfachId(MapUtils.getString(commandBody, PostfachNachricht.FIELD_POSTFACH_ID))
.replyOption(PostfachNachricht.ReplyOption.valueOf(MapUtils.getString(commandBody, PostfachNachricht.FIELD_REPLY_OPTION)))
.createdBy(command.getCreatedBy())
.direction(Direction.OUT)
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 f0b40d4b0da955f55c627a4edaa19ad7eb6e132e..a80d5b5bcad1b702d11b03738490ff5577661c7e 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
@@ -76,7 +76,6 @@ public abstract class PostfachNachrichtMapper {
.setId(MapUtils.getString(mailMap, PostfachNachricht.FIELD_ID))
.setVorgangId(MapUtils.getString(mailMap, PostfachNachricht.FIELD_VORGANG_ID))
.setReferencedNachricht(MapUtils.getString(mailMap, PostfachNachricht.REFERENCED_NACHRICHT_FIELD, StringUtils.EMPTY))
- .setPostfachId(MapUtils.getString(mailMap, PostfachNachricht.FIELD_POSTFACH_ID, StringUtils.EMPTY))
.setCreatedAt(MapUtils.getString(mailMap, PostfachNachricht.FIELD_CREATED_AT))
.setCreatedBy(MapUtils.getString(mailMap, PostfachNachricht.FIELD_CREATED_BY, StringUtils.EMPTY))
.setSentAt(MapUtils.getString(mailMap, PostfachNachricht.FIELD_SENT_AT, StringUtils.EMPTY))
@@ -131,7 +130,6 @@ public abstract class PostfachNachrichtMapper {
.id(MapUtils.getString(mailMap, PostfachNachricht.FIELD_ID))
.vorgangId(MapUtils.getString(mailMap, (PostfachNachricht.FIELD_VORGANG_ID)))
.referencedNachricht(MapUtils.getString(mailMap, PostfachNachricht.REFERENCED_NACHRICHT_FIELD, StringUtils.EMPTY))
- .postfachId(MapUtils.getString(mailMap, PostfachNachricht.FIELD_POSTFACH_ID, StringUtils.EMPTY))
.messageId(MapUtils.getString(mailMap, PostfachNachricht.FIELD_MESSAGE_ID))
.createdAt(ZonedDateTime.parse(MapUtils.getString(mailMap, PostfachNachricht.FIELD_CREATED_AT)))
.createdBy(MapUtils.getString(mailMap, PostfachNachricht.FIELD_CREATED_BY))
@@ -202,7 +200,6 @@ public abstract class PostfachNachrichtMapper {
PostfachNachricht.FIELD_MAIL_BODY, nachricht.getMailBody(),
PostfachNachricht.FIELD_ATTACHMENTS, nachricht.getAttachments()));
- putIfNonNull(result, PostfachNachricht.FIELD_POSTFACH_ID, nachricht.getPostfachId());
putIfNonNull(result, PostfachNachricht.FIELD_MESSAGE_ID, nachricht.getMessageId());
putIfNonNull(result, PostfachNachricht.FIELD_CREATED_BY, nachricht.getCreatedBy());
putIfNonNull(result, PostfachNachricht.FIELD_SENT_SUCCESSFUL, nachricht.getSentSuccessful());
diff --git a/nachrichten-manager-server/src/main/java/de/ozgcloud/nachrichten/postfach/osi/OsiPostfachMessageMapper.java b/nachrichten-manager-server/src/main/java/de/ozgcloud/nachrichten/postfach/osi/OsiPostfachMessageMapper.java
index ad900a06ffc7f7a077444f70276d627d2be8a657..38c7c4a73587a9ab0c4ac3371f62203f59d77fab 100644
--- a/nachrichten-manager-server/src/main/java/de/ozgcloud/nachrichten/postfach/osi/OsiPostfachMessageMapper.java
+++ b/nachrichten-manager-server/src/main/java/de/ozgcloud/nachrichten/postfach/osi/OsiPostfachMessageMapper.java
@@ -3,6 +3,7 @@ package de.ozgcloud.nachrichten.postfach.osi;
import java.util.List;
import java.util.Optional;
+import org.apache.commons.lang3.StringUtils;
import org.mapstruct.Mapper;
import org.mapstruct.Mapping;
import org.mapstruct.Named;
@@ -51,15 +52,13 @@ public abstract class OsiPostfachMessageMapper {
return attachments.stream().map(attachment -> messageAttachmentService.persistAttachment(vorgangId, attachment)).toList();
}
- @Mapping(target = "attachment", ignore = true)
- @Mapping(target = "messageId", ignore = true)
String toPostfachId(PostfachNachricht nachricht) {
return Optional.ofNullable(nachricht.getPostfachAddress())
.map(PostfachAddress::getIdentifier)
.filter(StringBasedIdentifier.class::isInstance)
.map(StringBasedIdentifier.class::cast)
.map(StringBasedIdentifier::getPostfachId)
- .orElse(nachricht.getPostfachId());
+ .orElse(StringUtils.EMPTY);
}
PostfachAddress buildPostfachAddressByPostfachId(String postfachId) {
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 5ad3dbcac003e9bfa6120fad4a40c41382bae03c..5c80887c39383eb6a4c8d935ce7864415344233b 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
@@ -110,25 +110,6 @@ class PostfachEventListenerTest {
assertThat(nachrichtCaptor.getValue().getId()).isEqualTo(PostfachNachrichtTestFactory.ID);
}
- @DisplayName("with postfachAddress")
- @Nested
- class TestWithPostfachAddress {
-
- @BeforeEach
- void mockCommandBodyObject() {
- var body = buildCommandBody();
- body.remove(PostfachNachricht.FIELD_POSTFACH_ID);
- when(command.getBodyObject()).thenReturn(body);
- }
-
- @Test
- void shouldHandleEmptyPostfachId() {
- var postfachMailToSend = sendPostfachNachricht();
-
- assertThat(postfachMailToSend.getPostfachId()).isNull();
- }
- }
-
@DisplayName("with postfachId")
@Nested
class TestWithPostfachId {
diff --git a/nachrichten-manager-server/src/test/java/de/ozgcloud/nachrichten/postfach/PostfachGrpcServiceTest.java b/nachrichten-manager-server/src/test/java/de/ozgcloud/nachrichten/postfach/PostfachGrpcServiceTest.java
index 163cf3ac79d6d890cb8796b94d10bdf270d30400..ae6d5b59be186f5d2fd0c0a649881e0354a136f3 100644
--- a/nachrichten-manager-server/src/test/java/de/ozgcloud/nachrichten/postfach/PostfachGrpcServiceTest.java
+++ b/nachrichten-manager-server/src/test/java/de/ozgcloud/nachrichten/postfach/PostfachGrpcServiceTest.java
@@ -40,7 +40,6 @@ import org.mockito.InjectMocks;
import org.mockito.Mock;
import org.mockito.Spy;
-import de.ozgcloud.nachrichten.postfach.osi.MessageTestFactory;
import io.grpc.stub.StreamObserver;
class PostfachGrpcServiceTest {
@@ -129,7 +128,6 @@ class PostfachGrpcServiceTest {
private PostfachNachricht createPostfachNachricht() {
return PostfachNachrichtTestFactory.createBuilder()
.id(GrpcPostfachMailTestFactory.ID)
- .postfachId(GrpcPostfachMailTestFactory.POSTFACH_ID)
.vorgangId(GrpcPostfachMailTestFactory.VORGANG_ID)
.postfachAddress(PostfachAddressTestFactory.createBuilder()
.identifier(StringBasedIdentifier.builder().postfachId(GrpcPostfachMailTestFactory.POSTFACH_ID).build()).build())
@@ -282,7 +280,7 @@ class PostfachGrpcServiceTest {
@Nested
class TestGetPostfachConfig {
- private GrpcGetPostfachConfigRequest request = GrpcGetPostfachConfigRequest.newBuilder().build();
+ private final GrpcGetPostfachConfigRequest request = GrpcGetPostfachConfigRequest.newBuilder().build();
@Mock
private StreamObserver<GrpcGetPostfachConfigResponse> responseObserver;
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 40f9862129b74ab2813d04e82cdcac2d398411b1..db35ab2c07471d1c43760cb7eab0cb2563ec6afb 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
@@ -37,8 +37,6 @@ import org.junit.jupiter.api.BeforeEach;
import org.junit.jupiter.api.DisplayName;
import org.junit.jupiter.api.Nested;
import org.junit.jupiter.api.Test;
-import org.junit.jupiter.params.ParameterizedTest;
-import org.junit.jupiter.params.provider.ValueSource;
import org.mapstruct.factory.Mappers;
import org.mockito.InjectMocks;
import org.mockito.Mock;
@@ -109,15 +107,6 @@ class PostfachNachrichtMapperTest {
assertThat(grpcMail.getSentAt()).isEmpty();
}
- @ParameterizedTest
- @ValueSource(strings = PostfachNachricht.FIELD_POSTFACH_ID)
- void shouldMapDefaultOnNonExistingValues(String fieldName) {
- var mapped = fromMap(getMapWithoutProperty(fieldName));
-
- var fieldValue = getAttributeFromGrpcObject(mapped, fieldName);
- assertThat((String) fieldValue).isEmpty();
- }
-
@Test
void shouldProceedWithSingleAttachment() {
var postfachNachrichtMap = PostfachNachrichtTestFactory.asMap();
@@ -214,15 +203,6 @@ class PostfachNachrichtMapperTest {
}
}
- @ParameterizedTest
- @ValueSource(strings = PostfachNachricht.FIELD_POSTFACH_ID)
- void shouldMapDefaultOnNonExistingValues(String fieldName) {
- var mapped = fromMapToPostfachMail(getMapWithoutProperty(fieldName));
-
- var fieldValue = getAttributeFromObject(mapped, fieldName);
- assertThat((String) fieldValue).isEmpty();
- }
-
private PostfachNachricht fromMapToPostfachMail(Map<String, Object> map) {
return mapper.fromMapToPostfachMail(map);
}
@@ -288,13 +268,12 @@ class PostfachNachrichtMapperTest {
.setMessageCode(PostfachNachrichtTestFactory.MESSAGE_CODE)
.setPostfachAddress(GrpcPostfachAddressTestFactory.createBuilder()
.setIdentifier(GrpcPostfachAddressTestFactory.createIdentifier(MessageTestFactory.POSTFACH_ID)))
- .setPostfachId(MessageTestFactory.POSTFACH_ID)
.setVorgangId(MessageTestFactory.VORGANG_ID).build();
}
@Test
void shouldHandleEmptyPostfachId() {
- var postfachMailWithoutPostfachId = GrpcPostfachMailTestFactory.createBuilder().clearPostfachId().build();
+ var postfachMailWithoutPostfachId = GrpcPostfachMailTestFactory.createBuilder().build();
assertDoesNotThrow(() -> fromGrpcMail(postfachMailWithoutPostfachId));
}
@@ -354,7 +333,7 @@ class PostfachNachrichtMapperTest {
@Test
void shouldHandleNullAsPostfachId() {
- var postfachMail = PostfachNachrichtTestFactory.createBuilder().postfachId(null).build();
+ var postfachMail = PostfachNachrichtTestFactory.create();
assertDoesNotThrow(() -> mapper.mapToMap(postfachMail));
}
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 d58ab79a63e7b6984c5c9d641bef9460086e2117..cca798eee34909ce1ae5d6ae499b9a12e4362045 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
@@ -61,7 +61,6 @@ public class PostfachNachrichtTestFactory {
return PostfachNachricht.builder()
.id(ID)
.referencedNachricht(GrpcPostfachMailTestFactory.REFERENCED_NACHRICHT_ID)
- .postfachId(POSTFACH_ID)
.postfachAddress(PostfachAddressTestFactory.create())
.messageId(MESSAGE_ID)
.vorgangId(VORGANG_ID)
@@ -81,7 +80,6 @@ public class PostfachNachrichtTestFactory {
var map = new HashMap<String, Object>();
map.put(PostfachNachricht.FIELD_ID, ID);
map.put(PostfachNachricht.REFERENCED_NACHRICHT_FIELD, GrpcPostfachMailTestFactory.REFERENCED_NACHRICHT_ID);
- map.put(PostfachNachricht.FIELD_POSTFACH_ID, POSTFACH_ID);
map.put(PostfachNachricht.POSTFACH_ADDRESS_FIELD, getPostfachAddressAsMap());
map.put(PostfachNachricht.FIELD_MESSAGE_ID, MESSAGE_ID);
map.put(PostfachNachricht.FIELD_VORGANG_ID, VORGANG_ID);
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 03296b4b114b2e57e1b4e6158f9cd09ca50cbbd3..acbf9bdae93369065186749cd2c2361a1e828d96 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
@@ -172,10 +172,10 @@ class OsiPostfachMessageMapperTest {
class TestWithoutPostfachAddress {
@Test
- void shouldMapPostfachAddressToPostfachId() {
+ void shouldMapPostfachAddressToEmptyString() {
var message = toOsiMessage(PostfachNachrichtTestFactory.createBuilder().postfachAddress(null).build());
- assertThat(message.getPostfachId()).isEqualTo(MessageTestFactory.POSTFACH_ID);
+ assertThat(message.getPostfachId()).isEmpty();
}
}
}