diff --git a/alfa-service/src/main/java/de/ozgcloud/alfa/postfach/PostfachMailModelAssembler.java b/alfa-service/src/main/java/de/ozgcloud/alfa/postfach/PostfachMailModelAssembler.java index 9efceff0fd017b2387d7440769a6dba41a6772c5..6d73eb8ef1765d444249ff763ae7d1a73789f30a 100644 --- a/alfa-service/src/main/java/de/ozgcloud/alfa/postfach/PostfachMailModelAssembler.java +++ b/alfa-service/src/main/java/de/ozgcloud/alfa/postfach/PostfachMailModelAssembler.java @@ -90,7 +90,7 @@ class PostfachMailModelAssembler implements RepresentationModelAssembler<Postfac var model = buildHalRepresentationModel(postfachMailsList.stream(), vorgang, postfachSettings); if (hasServiceKonto(vorgang)) { - addPostfachNachrichtLinks(model, getVorgangInfo(postfachMailsList, vorgang)); + addPostfachNachrichtLinks(model, buildVorgangMailInfo(postfachMailsList, vorgang)); } return model; } @@ -108,17 +108,15 @@ class PostfachMailModelAssembler implements RepresentationModelAssembler<Postfac return Optional.ofNullable(vorgang.getHeader()).map(VorgangHead::getServiceKonto).isPresent(); } - record VorgangInfo(VorgangWithEingang vorgang, boolean hasIncomingMails) {} - - VorgangInfo getVorgangInfo(List<PostfachMail> postfachMails, VorgangWithEingang vorgang) { - return new VorgangInfo(vorgang, hasIncomingMails(postfachMails)); + VorgangMailInfo buildVorgangMailInfo(List<PostfachMail> postfachMails, VorgangWithEingang vorgang) { + return new VorgangMailInfo(vorgang, hasIncomingMails(postfachMails)); } boolean hasIncomingMails(List<PostfachMail> postfachMails) { return postfachMails.stream().anyMatch(IS_INCOMING); } - void addPostfachNachrichtLinks(RepresentationModel<EntityModel<PostfachSettings>> model, VorgangInfo vorgangInfo) { + void addPostfachNachrichtLinks(RepresentationModel<EntityModel<PostfachSettings>> model, VorgangMailInfo vorgangInfo) { var vorgangId = vorgangInfo.vorgang.getId(); if (vorgangController.isEditable(vorgangInfo.vorgang)) { @@ -156,4 +154,5 @@ class PostfachMailModelAssembler implements RepresentationModelAssembler<Postfac () -> Link.of(String.format(userManagerUrlProvider.getUserProfileTemplate(), postfachMail.getCreatedBy()), REL_CREATED_BY)); } + record VorgangMailInfo(VorgangWithEingang vorgang, boolean hasIncomingMails) {} } \ No newline at end of file diff --git a/alfa-service/src/test/java/de/ozgcloud/alfa/postfach/PostfachMailModelAssemblerTest.java b/alfa-service/src/test/java/de/ozgcloud/alfa/postfach/PostfachMailModelAssemblerTest.java index 178be8905d2d50a909dd95efa4e2c0362a0f4d68..da9a28e8d99613687b9d6315c3c515d40c4d8281 100644 --- a/alfa-service/src/test/java/de/ozgcloud/alfa/postfach/PostfachMailModelAssemblerTest.java +++ b/alfa-service/src/test/java/de/ozgcloud/alfa/postfach/PostfachMailModelAssemblerTest.java @@ -58,7 +58,7 @@ import de.ozgcloud.alfa.common.user.UserId; import de.ozgcloud.alfa.common.user.UserManagerUrlProvider; import de.ozgcloud.alfa.postfach.PostfachMail.Direction; import de.ozgcloud.alfa.postfach.PostfachMailController.PostfachMailCommandByVorgangController; -import de.ozgcloud.alfa.postfach.PostfachMailModelAssembler.VorgangInfo; +import de.ozgcloud.alfa.postfach.PostfachMailModelAssembler.VorgangMailInfo; import de.ozgcloud.alfa.vorgang.ServiceKontoTestFactory; import de.ozgcloud.alfa.vorgang.VorgangController; import de.ozgcloud.alfa.vorgang.VorgangHeadTestFactory; @@ -324,18 +324,18 @@ class PostfachMailModelAssemblerTest { @Nested class OnHasServiceKonto { - private final VorgangInfo vorgangInfo = new VorgangInfo(vorgang, true); + private final VorgangMailInfo vorgangMailInfo = new VorgangMailInfo(vorgang, true); @BeforeEach void init() { - doReturn(vorgangInfo).when(modelAssembler).getVorgangInfo(any(), any()); + doReturn(vorgangMailInfo).when(modelAssembler).buildVorgangMailInfo(any(), any()); } @Test - void shouldGetVorgangInfo() { + void shouldBuildVorgangMailInfo() { callModelAssembler(); - verify(modelAssembler).getVorgangInfo(mails, vorgang); + verify(modelAssembler).buildVorgangMailInfo(mails, vorgang); } @Test @@ -344,7 +344,7 @@ class PostfachMailModelAssemblerTest { callModelAssembler(); - verify(modelAssembler).addPostfachNachrichtLinks(model, vorgangInfo); + verify(modelAssembler).addPostfachNachrichtLinks(model, vorgangMailInfo); } } @@ -438,14 +438,14 @@ class PostfachMailModelAssemblerTest { } @Nested - class TestGetVorgangInfo { + class TestBuildVorgangMailInfo { private final List<PostfachMail> mails = List.of(PostfachMailTestFactory.create()); private final VorgangWithEingang vorgang = VorgangWithEingangTestFactory.create(); @Test void shouldCallHasIncomingMails() { - modelAssembler.getVorgangInfo(mails, vorgang); + modelAssembler.buildVorgangMailInfo(mails, vorgang); verify(modelAssembler).hasIncomingMails(mails); } @@ -455,16 +455,16 @@ class PostfachMailModelAssemblerTest { void shouldHaveHasIncomingMails(boolean hasIncomingMails) { doReturn(hasIncomingMails).when(modelAssembler).hasIncomingMails(any()); - var vorgangInfo = modelAssembler.getVorgangInfo(mails, vorgang); + var vorgangMailInfo = modelAssembler.buildVorgangMailInfo(mails, vorgang); - assertThat(vorgangInfo.hasIncomingMails()).isEqualTo(hasIncomingMails); + assertThat(vorgangMailInfo.hasIncomingMails()).isEqualTo(hasIncomingMails); } @Test void shouldHaveVorgang() { - var vorgangInfo = modelAssembler.getVorgangInfo(mails, vorgang); + var vorgangMailInfo = modelAssembler.buildVorgangMailInfo(mails, vorgang); - assertThat(vorgangInfo.vorgang()).isEqualTo(vorgang); + assertThat(vorgangMailInfo.vorgang()).isEqualTo(vorgang); } } @@ -571,7 +571,7 @@ class PostfachMailModelAssemblerTest { private void callModelAssembler() { modelAssembler.addPostfachNachrichtLinks(model, - new VorgangInfo(VorgangWithEingangTestFactory.createBuilder().hasNewPostfachNachricht(true).build(), true)); + new VorgangMailInfo(VorgangWithEingangTestFactory.createBuilder().hasNewPostfachNachricht(true).build(), true)); } } @@ -599,7 +599,7 @@ class PostfachMailModelAssemblerTest { private void callModelAssembler(boolean hasIncomingMails) { modelAssembler.addPostfachNachrichtLinks(model, - new VorgangInfo(VorgangWithEingangTestFactory.createBuilder().hasNewPostfachNachricht(false).build(), hasIncomingMails)); + new VorgangMailInfo(VorgangWithEingangTestFactory.createBuilder().hasNewPostfachNachricht(false).build(), hasIncomingMails)); } } } @@ -656,7 +656,7 @@ class PostfachMailModelAssemblerTest { } private void callModelAssembler(VorgangWithEingang vorgang) { - modelAssembler.addPostfachNachrichtLinks(model, new VorgangInfo(vorgang, false)); + modelAssembler.addPostfachNachrichtLinks(model, new VorgangMailInfo(vorgang, false)); } } }