diff --git a/nachrichten-manager-server/src/main/java/de/ozgcloud/nachrichten/antragraum/AntragraumGrpcService.java b/nachrichten-manager-server/src/main/java/de/ozgcloud/nachrichten/antragraum/AntragraumGrpcService.java index 8d5a6cf2d8d6a95947ca713818d571219db3ff68..60fb62d6bfa8384eb093780e82411474bec22373 100644 --- a/nachrichten-manager-server/src/main/java/de/ozgcloud/nachrichten/antragraum/AntragraumGrpcService.java +++ b/nachrichten-manager-server/src/main/java/de/ozgcloud/nachrichten/antragraum/AntragraumGrpcService.java @@ -70,14 +70,14 @@ class AntragraumGrpcService extends AntragraumServiceGrpc.AntragraumServiceImplB streamObserver.onCompleted(); } - Rueckfrage getRueckfrageByNachricht(String samlToken, String postfachNachrichtId) { + RueckfrageHead getRueckfrageByNachricht(String samlToken, String postfachNachrichtId) { return service.findRueckfragen(samlToken) .filter(rueckfrage -> StringUtils.equals(rueckfrage.getId(), postfachNachrichtId)) .findFirst() .orElseThrow(() -> new TechnicalException(String.format("No rueckfrage found for answer with id %s", postfachNachrichtId))); } - PostfachNachricht enrichRueckfrageAnswer(PostfachNachricht answer, Rueckfrage rueckfrage) { + PostfachNachricht enrichRueckfrageAnswer(PostfachNachricht answer, RueckfrageHead rueckfrage) { return answer.toBuilder() .vorgangId(rueckfrage.getVorgangId()) .postfachAddress(rueckfrage.getPostfachAddress()) 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 5bc08189c9903bb495e10b8f640f5331180e07bb..76594d789a6a10cded4ef88f5160b21d21218dc9 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 @@ -53,7 +53,7 @@ interface AntragraumNachrichtMapper { @Mapping(target = "vorgangNameBytes", ignore = true) @Mapping(target = "allFields", ignore = true) @Mapping(target = "sentAt", source = "sentAt", dateFormat = "yyyy-MM-dd'T'HH:mm:ss") - GrpcRueckfrageHead toGrpc(Rueckfrage rueckfrage); + GrpcRueckfrageHead toGrpc(RueckfrageHead rueckfrage); @Mapping(target = "referencedNachricht", ignore = true) @Mapping(target = "mailBody", source = "answerText") diff --git a/nachrichten-manager-server/src/main/java/de/ozgcloud/nachrichten/antragraum/AntragraumService.java b/nachrichten-manager-server/src/main/java/de/ozgcloud/nachrichten/antragraum/AntragraumService.java index 8d435066a9ce498fadcda5aa741119fff7d2b799..b7ae839cb454017adc22bd543228b1afe1d1e0da 100644 --- a/nachrichten-manager-server/src/main/java/de/ozgcloud/nachrichten/antragraum/AntragraumService.java +++ b/nachrichten-manager-server/src/main/java/de/ozgcloud/nachrichten/antragraum/AntragraumService.java @@ -101,7 +101,7 @@ public class AntragraumService { return USER_NOTIFICATION_TEMPLATE.formatted(getAntragsraumUrl()); } - public Stream<Rueckfrage> findRueckfragen(String samlToken) { + public Stream<RueckfrageHead> findRueckfragen(String samlToken) { verifyToken(samlToken); var rueckfragen = postfachNachrichtService.findRueckfragen(BAYERN_ID_SERVICE_KONTO_TYPE, getPostfachId(samlToken)) @@ -110,7 +110,7 @@ public class AntragraumService { return enrichRueckfragen(rueckfragen, samlToken); } - Rueckfrage enrichByAnswers(Rueckfrage rueckfrage, String samlToken) { + RueckfrageHead enrichByAnswers(RueckfrageHead rueckfrage, String samlToken) { var rueckfrageBuilder = rueckfrage.toBuilder(); getFirstSentAt(findAnswers(samlToken, rueckfrage.getId())) @@ -123,7 +123,7 @@ public class AntragraumService { return answers.sorted(Comparator.comparing(PostfachNachricht::getSentAt)).map(PostfachNachricht::getSentAt).findFirst(); } - Stream<Rueckfrage> enrichRueckfragen(List<Rueckfrage> rueckfragen, String samlToken) { + Stream<RueckfrageHead> enrichRueckfragen(List<RueckfrageHead> rueckfragen, String samlToken) { if (rueckfragen.isEmpty()) { return Stream.empty(); } @@ -134,11 +134,11 @@ public class AntragraumService { .map(rueckfrage -> addAccessible(rueckfrage, samlToken)); } - Rueckfrage addAccessible(Rueckfrage rueckfrage, String samlToken) { + RueckfrageHead addAccessible(RueckfrageHead rueckfrage, String samlToken) { return rueckfrage.toBuilder().accessible(isAccessible(samlToken, rueckfrage.getTrustLevel())).build(); } - Rueckfrage addVorgangData(Rueckfrage rueckfrage, Vorgang vorgang) { + RueckfrageHead addVorgangData(RueckfrageHead rueckfrage, Vorgang vorgang) { return rueckfrage.toBuilder() .vorgangName(vorgang.getName()) .vorgangNummer(vorgang.getNummer()) diff --git a/nachrichten-manager-server/src/main/java/de/ozgcloud/nachrichten/antragraum/Rueckfrage.java b/nachrichten-manager-server/src/main/java/de/ozgcloud/nachrichten/antragraum/RueckfrageHead.java similarity index 96% rename from nachrichten-manager-server/src/main/java/de/ozgcloud/nachrichten/antragraum/Rueckfrage.java rename to nachrichten-manager-server/src/main/java/de/ozgcloud/nachrichten/antragraum/RueckfrageHead.java index a398a46888cc5d12467015d68b350384e06e4c2a..8d7e25b27bd41c144bc3b107e3cf46a444228236 100644 --- a/nachrichten-manager-server/src/main/java/de/ozgcloud/nachrichten/antragraum/Rueckfrage.java +++ b/nachrichten-manager-server/src/main/java/de/ozgcloud/nachrichten/antragraum/RueckfrageHead.java @@ -8,7 +8,7 @@ import lombok.Getter; @Getter @Builder(toBuilder = true) -class Rueckfrage { +class RueckfrageHead { private String id; private String vorgangId; diff --git a/nachrichten-manager-server/src/main/java/de/ozgcloud/nachrichten/antragraum/RueckfrageMapper.java b/nachrichten-manager-server/src/main/java/de/ozgcloud/nachrichten/antragraum/RueckfrageMapper.java index f2f763430a497d3fb9c605313eb0939f7f55d9a1..b2a7ffddae6b391fab7c684b808f68d2b9972250 100644 --- a/nachrichten-manager-server/src/main/java/de/ozgcloud/nachrichten/antragraum/RueckfrageMapper.java +++ b/nachrichten-manager-server/src/main/java/de/ozgcloud/nachrichten/antragraum/RueckfrageMapper.java @@ -27,7 +27,7 @@ interface RueckfrageMapper { @Mapping(target = "answeredAt", ignore = true) @Mapping(target = "status", ignore = true) @Mapping(target = "vorgangName", ignore = true) - Rueckfrage fromPostfachNachricht(PostfachNachricht postfachNachricht); + RueckfrageHead fromPostfachNachricht(PostfachNachricht postfachNachricht); @Mapping(target = "mergeFrom", ignore = true) @Mapping(target = "clearField", ignore = true) diff --git a/nachrichten-manager-server/src/test/java/de/ozgcloud/nachrichten/antragraum/AntragraumGrpcServiceTest.java b/nachrichten-manager-server/src/test/java/de/ozgcloud/nachrichten/antragraum/AntragraumGrpcServiceTest.java index 5d6c596f6213b94a982fe439233befc1189188a4..a3815062085899da5db72cba9430d03f317a8c93 100644 --- a/nachrichten-manager-server/src/test/java/de/ozgcloud/nachrichten/antragraum/AntragraumGrpcServiceTest.java +++ b/nachrichten-manager-server/src/test/java/de/ozgcloud/nachrichten/antragraum/AntragraumGrpcServiceTest.java @@ -75,7 +75,7 @@ class AntragraumGrpcServiceTest { @Mock private StreamObserver<GrpcFindRueckfragenResponse> streamObserver; - private final Rueckfrage rueckfrage = RueckfrageTestFactory.create(); + private final RueckfrageHead rueckfrage = RueckfrageHeadTestFactory.create(); private final GrpcRueckfrageHead grpcRueckfrageHead = GrpcRueckfrageHeadTestFactory.create(); @@ -114,7 +114,7 @@ class AntragraumGrpcServiceTest { @Mock private StreamObserver<GrpcSendRueckfrageAnswerResponse> streamObserver; private PostfachNachricht postfachNachricht = PostfachNachrichtTestFactory.create(); - private Rueckfrage rueckfrage = RueckfrageTestFactory.create(); + private RueckfrageHead rueckfrage = RueckfrageHeadTestFactory.create(); @BeforeEach void setup() { @@ -177,9 +177,9 @@ class AntragraumGrpcServiceTest { @Nested class TestGetRueckfrageByNachricht { - private final Rueckfrage matchingRueckfrage = RueckfrageTestFactory.createBuilder().id(GrpcRueckfrageAnswerTestFactory.RUECKFRAGE_ID) + private final RueckfrageHead matchingRueckfrage = RueckfrageHeadTestFactory.createBuilder().id(GrpcRueckfrageAnswerTestFactory.RUECKFRAGE_ID) .build(); - private final Rueckfrage rueckfrage = RueckfrageTestFactory.create(); + private final RueckfrageHead rueckfrage = RueckfrageHeadTestFactory.create(); @BeforeEach void mock() { @@ -207,7 +207,7 @@ class AntragraumGrpcServiceTest { assertThatThrownBy(() -> getRueckfrage()).isInstanceOf(TechnicalException.class); } - private Rueckfrage getRueckfrage() { + private RueckfrageHead getRueckfrage() { return grpcService.getRueckfrageByNachricht(GrpcSendRueckfrageAnswerRequestTestFactory.SAML_TOKEN, GrpcRueckfrageAnswerTestFactory.RUECKFRAGE_ID); } @@ -223,7 +223,7 @@ class AntragraumGrpcServiceTest { .sentAt(null) .build(); - private final Rueckfrage postfachNachrichtRueckfrage = RueckfrageTestFactory.create(); + private final RueckfrageHead postfachNachrichtRueckfrage = RueckfrageHeadTestFactory.create(); @Test void shouldSetVorgangId() { diff --git a/nachrichten-manager-server/src/test/java/de/ozgcloud/nachrichten/antragraum/AntragraumNachrichtMapperTest.java b/nachrichten-manager-server/src/test/java/de/ozgcloud/nachrichten/antragraum/AntragraumNachrichtMapperTest.java index 8ac6a1e3ddba299a018cc77f314501b190cc19d9..aa1755d285bad777fae9b170b377c7b91687e3b8 100644 --- a/nachrichten-manager-server/src/test/java/de/ozgcloud/nachrichten/antragraum/AntragraumNachrichtMapperTest.java +++ b/nachrichten-manager-server/src/test/java/de/ozgcloud/nachrichten/antragraum/AntragraumNachrichtMapperTest.java @@ -47,21 +47,21 @@ class AntragraumNachrichtMapperTest { void shouldMapVorgangId() { var result = map(); - assertThat(result.getVorgangId()).isEqualTo(RueckfrageTestFactory.VORGANG_ID); + assertThat(result.getVorgangId()).isEqualTo(RueckfrageHeadTestFactory.VORGANG_ID); } @Test void shouldMapId() { var result = map(); - assertThat(result.getId()).isEqualTo(RueckfrageTestFactory.ID); + assertThat(result.getId()).isEqualTo(RueckfrageHeadTestFactory.ID); } @Test void shouldMapVorgangName() { var result = map(); - assertThat(result.getVorgangName()).isEqualTo(RueckfrageTestFactory.VORGANG_NAME); + assertThat(result.getVorgangName()).isEqualTo(RueckfrageHeadTestFactory.VORGANG_NAME); } @Test @@ -79,7 +79,7 @@ class AntragraumNachrichtMapperTest { } private GrpcRueckfrageHead map() { - return mapper.toGrpc(RueckfrageTestFactory.create()); + return mapper.toGrpc(RueckfrageHeadTestFactory.create()); } } diff --git a/nachrichten-manager-server/src/test/java/de/ozgcloud/nachrichten/antragraum/AntragraumServiceTest.java b/nachrichten-manager-server/src/test/java/de/ozgcloud/nachrichten/antragraum/AntragraumServiceTest.java index ae98c89690880467a766152682096025bd4f2259..1b0bd3f3c2168126a21ebdb75a118626394eaf99 100644 --- a/nachrichten-manager-server/src/test/java/de/ozgcloud/nachrichten/antragraum/AntragraumServiceTest.java +++ b/nachrichten-manager-server/src/test/java/de/ozgcloud/nachrichten/antragraum/AntragraumServiceTest.java @@ -81,7 +81,7 @@ class AntragraumServiceTest { private static final String SAML_TOKEN = "TOKEN"; private static final String POSTFACH_ID = "dummyPostfachId"; - private final Rueckfrage rueckfrage = RueckfrageTestFactory.create(); + private final RueckfrageHead rueckfrage = RueckfrageHeadTestFactory.create(); private final PostfachNachricht postfachNachricht = PostfachNachrichtTestFactory.create(); @BeforeEach @@ -145,7 +145,7 @@ class AntragraumServiceTest { assertThat(rueckfragen).containsExactly(rueckfrage); } - private List<Rueckfrage> findRueckfragen() { + private List<RueckfrageHead> findRueckfragen() { return service.findRueckfragen(SAML_TOKEN).toList(); } } @@ -155,7 +155,7 @@ class AntragraumServiceTest { class TestEnrichByAnswers { private static final String SAML_TOKEN = "TOKEN"; - private final Rueckfrage rueckfrage = RueckfrageTestFactory.create(); + private final RueckfrageHead rueckfrage = RueckfrageHeadTestFactory.create(); private final PostfachNachricht postfachNachricht = PostfachNachrichtTestFactory.create(); private final Stream<PostfachNachricht> answers = Stream.of(postfachNachricht); @@ -169,7 +169,7 @@ class AntragraumServiceTest { void shouldCallFindAnswers() { enrichByAnswer(); - verify(service).findAnswers(SAML_TOKEN, RueckfrageTestFactory.ID); + verify(service).findAnswers(SAML_TOKEN, RueckfrageHeadTestFactory.ID); } @Test @@ -194,7 +194,7 @@ class AntragraumServiceTest { .isEqualTo(rueckfrage); } - private Rueckfrage enrichByAnswer() { + private RueckfrageHead enrichByAnswer() { return service.enrichByAnswers(rueckfrage, SAML_TOKEN); } } @@ -203,8 +203,8 @@ class AntragraumServiceTest { @Nested class TestEnrichRueckfragen { - private final Rueckfrage rueckfrage = RueckfrageTestFactory.create(); - private final List<Rueckfrage> rueckfragen = Collections.singletonList(rueckfrage); + private final RueckfrageHead rueckfrageHead = RueckfrageHeadTestFactory.create(); + private final List<RueckfrageHead> rueckfragen = Collections.singletonList(rueckfrageHead); private final Vorgang vorgang = VorgangTestFactory.create(); @@ -224,40 +224,40 @@ class AntragraumServiceTest { void mock() { when(vorgangService.getVorgang(any())).thenReturn(vorgang); - doReturn(rueckfrage).when(service).addVorgangData(any(), any()); - doReturn(rueckfrage).when(service).addAccessible(any(), any()); + doReturn(rueckfrageHead).when(service).addVorgangData(any(), any()); + doReturn(rueckfrageHead).when(service).addAccessible(any(), any()); } @Test void shouldCallVorgangService() { enrichRueckfragen(); - verify(vorgangService).getVorgang(RueckfrageTestFactory.VORGANG_ID); + verify(vorgangService).getVorgang(RueckfrageHeadTestFactory.VORGANG_ID); } @Test void shouldAddVorgangData() { enrichRueckfragen(); - verify(service).addVorgangData(rueckfrage, vorgang); + verify(service).addVorgangData(rueckfrageHead, vorgang); } @Test void shouldAddAccessible() { enrichRueckfragen(); - verify(service).addAccessible(rueckfrage, GrpcGetRueckfrageRequestTestFactory.SAML_TOKEN); + verify(service).addAccessible(rueckfrageHead, GrpcGetRueckfrageRequestTestFactory.SAML_TOKEN); } @Test void shouldReturnEnrichedRueckfragen() { var enrichedRueckfragen = enrichRueckfragen(); - assertThat(enrichedRueckfragen).containsExactly(rueckfrage); + assertThat(enrichedRueckfragen).containsExactly(rueckfrageHead); } } - private List<Rueckfrage> enrichRueckfragen() { + private List<RueckfrageHead> enrichRueckfragen() { return service.enrichRueckfragen(rueckfragen, GrpcGetRueckfrageRequestTestFactory.SAML_TOKEN).toList(); } } @@ -266,7 +266,7 @@ class AntragraumServiceTest { @Nested class TestAddAccessible { - private final Rueckfrage rueckfrage = RueckfrageTestFactory.createBuilder().accessible(false).build(); + private final RueckfrageHead rueckfrageHead = RueckfrageHeadTestFactory.createBuilder().accessible(false).build(); @BeforeEach void mock() { @@ -275,14 +275,14 @@ class AntragraumServiceTest { @Test void shouldCallIsAccessible() { - service.addAccessible(rueckfrage, GrpcGetRueckfrageRequestTestFactory.SAML_TOKEN); + service.addAccessible(rueckfrageHead, GrpcGetRueckfrageRequestTestFactory.SAML_TOKEN); verify(service).isAccessible(GrpcGetRueckfrageRequestTestFactory.SAML_TOKEN, GrpcServiceKontoTestFactory.TRUST_LEVEL); } @Test void shouldSetAccessible() { - var rueckfrageWithAccessible = service.addAccessible(rueckfrage, GrpcGetRueckfrageRequestTestFactory.SAML_TOKEN); + var rueckfrageWithAccessible = service.addAccessible(rueckfrageHead, GrpcGetRueckfrageRequestTestFactory.SAML_TOKEN); assertThat(rueckfrageWithAccessible.isAccessible()).isTrue(); } @@ -313,8 +313,8 @@ class AntragraumServiceTest { assertThat(rueckfragenWithVorgangData.getTrustLevel()).isEqualTo(GrpcServiceKontoTestFactory.TRUST_LEVEL); } - private Rueckfrage addVorgangData() { - return service.addVorgangData(Rueckfrage.builder().build(), VorgangTestFactory.create()); + private RueckfrageHead addVorgangData() { + return service.addVorgangData(RueckfrageHead.builder().build(), VorgangTestFactory.create()); } } @@ -448,7 +448,7 @@ class AntragraumServiceTest { void shouldCallPersistPostfachNachrichtService() { getRueckfrage(); - verify(postfachNachrichtService).getById(RueckfrageTestFactory.ID); + verify(postfachNachrichtService).getById(RueckfrageHeadTestFactory.ID); } @Test @@ -466,7 +466,7 @@ class AntragraumServiceTest { } private PostfachNachricht getRueckfrage() { - return service.getRueckfrage(samlToken, RueckfrageTestFactory.ID); + return service.getRueckfrage(samlToken, RueckfrageHeadTestFactory.ID); } } diff --git a/nachrichten-manager-server/src/test/java/de/ozgcloud/nachrichten/antragraum/RueckfrageTestFactory.java b/nachrichten-manager-server/src/test/java/de/ozgcloud/nachrichten/antragraum/RueckfrageHeadTestFactory.java similarity index 81% rename from nachrichten-manager-server/src/test/java/de/ozgcloud/nachrichten/antragraum/RueckfrageTestFactory.java rename to nachrichten-manager-server/src/test/java/de/ozgcloud/nachrichten/antragraum/RueckfrageHeadTestFactory.java index 884770cf07f3bac78505db98f1ac56e2f821bb03..e9405169cae3b2ee605ae346e7b212f14f0294ea 100644 --- a/nachrichten-manager-server/src/test/java/de/ozgcloud/nachrichten/antragraum/RueckfrageTestFactory.java +++ b/nachrichten-manager-server/src/test/java/de/ozgcloud/nachrichten/antragraum/RueckfrageHeadTestFactory.java @@ -8,18 +8,18 @@ import de.ozgcloud.nachrichten.common.vorgang.GrpcServiceKontoTestFactory; import de.ozgcloud.nachrichten.postfach.PostfachAddressTestFactory; import de.ozgcloud.nachrichten.postfach.PostfachNachrichtTestFactory; -public class RueckfrageTestFactory { +public class RueckfrageHeadTestFactory { public static final String ID = UUID.randomUUID().toString(); public static final String VORGANG_ID = UUID.randomUUID().toString(); public static final String VORGANG_NAME = LoremIpsum.getInstance().getName(); - public static Rueckfrage create() { + public static RueckfrageHead create() { return createBuilder().build(); } - public static Rueckfrage.RueckfrageBuilder createBuilder() { - return Rueckfrage.builder() + public static RueckfrageHead.RueckfrageHeadBuilder createBuilder() { + return RueckfrageHead.builder() .id(ID) .vorgangId(VORGANG_ID) .vorgangName(VORGANG_NAME) diff --git a/nachrichten-manager-server/src/test/java/de/ozgcloud/nachrichten/antragraum/RueckfrageMapperTest.java b/nachrichten-manager-server/src/test/java/de/ozgcloud/nachrichten/antragraum/RueckfrageMapperTest.java index 11f099d2c8fe331cc66de36f5103dd03692735d4..7fc8bf3676f32b4c24aeb8795e43ee06193bcb86 100644 --- a/nachrichten-manager-server/src/test/java/de/ozgcloud/nachrichten/antragraum/RueckfrageMapperTest.java +++ b/nachrichten-manager-server/src/test/java/de/ozgcloud/nachrichten/antragraum/RueckfrageMapperTest.java @@ -61,7 +61,7 @@ class RueckfrageMapperTest { assertThat(mapped.getStatus()).isEqualTo(RueckfrageStatus.NEW); } - private Rueckfrage mapFromPostfachNachricht() { + private RueckfrageHead mapFromPostfachNachricht() { return mapper.fromPostfachNachricht(PostfachNachrichtTestFactory.create()); } }