From 6d11d95d8d4b4ecbf3ab158d5f3ce140750ed516 Mon Sep 17 00:00:00 2001
From: OZGCloud <ozgcloud@mgm-tp.com>
Date: Mon, 7 Oct 2024 05:34:57 +0200
Subject: [PATCH] OZG-6522 remove deprecated field postfachId
---
.../src/main/protobuf/postfach.model.proto | 1 -
.../postfach/GrpcPostfachMailTestFactory.java | 1 -
.../postfach/PostfachNachricht.java | 2 --
.../antragraum/AntragraumNachrichtMapper.java | 1 -
.../postfach/GrpcPostfachNachrichtMapper.java | 1 -
.../postfach/PostfachEventListener.java | 1 -
.../postfach/PostfachNachrichtMapper.java | 3 ---
.../osi/OsiPostfachMessageMapper.java | 5 ++--
.../postfach/PostfachEventListenerTest.java | 19 --------------
.../postfach/PostfachGrpcServiceTest.java | 4 +--
.../postfach/PostfachNachrichtMapperTest.java | 25 ++-----------------
.../PostfachNachrichtTestFactory.java | 2 --
.../osi/OsiPostfachMessageMapperTest.java | 4 +--
13 files changed, 7 insertions(+), 62 deletions(-)
diff --git a/nachrichten-manager-interface/src/main/protobuf/postfach.model.proto b/nachrichten-manager-interface/src/main/protobuf/postfach.model.proto
index d3cfc10..b7f2a68 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 cb6eccf..c1cd027 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 d76a825..8117a94 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 3c14695..b41a700 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 53a1512..7f4cc63 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 9937bbb..7b4a3be 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 f0b40d4..a80d5b5 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 ad900a0..38c7c4a 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 5ad3dbc..5c80887 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 163cf3a..ae6d5b5 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 40f9862..db35ab2 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 d58ab79..cca798e 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 03296b4..acbf9bd 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();
}
}
}
--
GitLab