Skip to content
Snippets Groups Projects
Commit 33458b51 authored by Krzysztof Witukiewicz's avatar Krzysztof Witukiewicz
Browse files

OZG-7773 OZG-8079 Renamings

parent e2494757
No related branches found
No related tags found
1 merge request!29OZG-7773 setHasNewPostfachNachricht only if there are incoming mails
This commit is part of merge request !29. Comments created here will be created in the context of that merge request.
......@@ -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
......@@ -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));
}
}
}
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment