From 8e20f4e8bed7adc0605e00ec0f55587c429cb352 Mon Sep 17 00:00:00 2001
From: Felix Reichenbach <felix.reichenbach@mgm-tp.com>
Date: Tue, 11 Mar 2025 13:53:24 +0100
Subject: [PATCH] OZG-7572 apply code review comments
---
.../de/ozgcloud/alfa/RootViewLinkHandler.java | 14 ++++++-------
.../alfa/statistic/StatisticService.java | 8 ++++----
.../alfa/RootViewLinkHandlerTest.java | 20 +++++++++----------
3 files changed, 21 insertions(+), 21 deletions(-)
diff --git a/alfa-service/src/main/java/de/ozgcloud/alfa/RootViewLinkHandler.java b/alfa-service/src/main/java/de/ozgcloud/alfa/RootViewLinkHandler.java
index 99d9c9dffd..6b1eddb06b 100644
--- a/alfa-service/src/main/java/de/ozgcloud/alfa/RootViewLinkHandler.java
+++ b/alfa-service/src/main/java/de/ozgcloud/alfa/RootViewLinkHandler.java
@@ -70,6 +70,9 @@ class RootViewLinkHandler {
static final int PAGE_SIZE = 100;
+ private static final List<VorgangStatus> STATUS_LIST = List.of(VorgangStatus.NEU, VorgangStatus.ANGENOMMEN, VorgangStatus.IN_BEARBEITUNG,
+ VorgangStatus.BESCHIEDEN, VorgangStatus.ABGESCHLOSSEN, VorgangStatus.VERWORFEN, VorgangStatus.ZU_LOESCHEN, VorgangStatus.WEITERGELEITET);
+
private final CurrentUserService currentUserService;
private final SystemStatusService systemStatusService;
private final FeatureToggleProperties featureToggleProperties;
@@ -158,7 +161,7 @@ class RootViewLinkHandler {
}
void addVorgangStatusLinks(EntityModel<Root> model, String linkRelTemplate) {
- getVorgangStatusStream().forEach(status -> model.add(buildGetAllVorgaengeByStatus(status, linkRelTemplate)));
+ getVorgangStatuses().forEach(status -> model.add(buildGetAllVorgaengeByStatus(status, linkRelTemplate)));
}
Link buildGetAllVorgaengeByStatus(VorgangStatus status, String linkRelTemplate) {
@@ -175,14 +178,11 @@ class RootViewLinkHandler {
}
List<Link> buildVorgangStatusLinks(String linkRelTemplate, UserId userId) {
- return getVorgangStatusStream().map(status -> buildGetAllVorgaengeByAssignedToAndStatus(status, linkRelTemplate, userId)).toList();
+ return getVorgangStatuses().map(status -> buildGetAllVorgaengeByAssignedToAndStatus(status, linkRelTemplate, userId)).toList();
}
- Stream<VorgangStatus> getVorgangStatusStream() {
- return Stream
- .of(VorgangStatus.NEU, VorgangStatus.ANGENOMMEN, VorgangStatus.IN_BEARBEITUNG, VorgangStatus.BESCHIEDEN, VorgangStatus.ABGESCHLOSSEN,
- VorgangStatus.VERWORFEN, VorgangStatus.ZU_LOESCHEN, VorgangStatus.WEITERGELEITET)
- .filter(status -> status != VorgangStatus.WEITERGELEITET || featureToggleProperties.isForwardByOzgCloudEnabled());
+ Stream<VorgangStatus> getVorgangStatuses() {
+ return STATUS_LIST.stream().filter(status -> status != VorgangStatus.WEITERGELEITET || featureToggleProperties.isForwardByOzgCloudEnabled());
}
Link buildGetAllVorgaengeByAssignedToAndStatus(VorgangStatus status, String linkRelTemplate, UserId userId) {
diff --git a/alfa-service/src/main/java/de/ozgcloud/alfa/statistic/StatisticService.java b/alfa-service/src/main/java/de/ozgcloud/alfa/statistic/StatisticService.java
index 9710c03464..57b9c2e7c5 100644
--- a/alfa-service/src/main/java/de/ozgcloud/alfa/statistic/StatisticService.java
+++ b/alfa-service/src/main/java/de/ozgcloud/alfa/statistic/StatisticService.java
@@ -27,7 +27,6 @@ import java.util.Collections;
import java.util.List;
import java.util.Map;
import java.util.Optional;
-import java.util.stream.Stream;
import org.springframework.stereotype.Service;
@@ -41,6 +40,9 @@ import lombok.RequiredArgsConstructor;
@RequiredArgsConstructor
class StatisticService {
+ private static final List<VorgangStatus> COUNT_BY_VORGANG_STATUS_VERWALTUNG_USER = List.of(VorgangStatus.NEU, VorgangStatus.ANGENOMMEN,
+ VorgangStatus.IN_BEARBEITUNG, VorgangStatus.BESCHIEDEN, VorgangStatus.ABGESCHLOSSEN, VorgangStatus.VERWORFEN, VorgangStatus.ZU_LOESCHEN,
+ VorgangStatus.WEITERGELEITET);
private static final List<VorgangStatus> COUNT_BY_VORGANG_STATUS_POSTSTELLE_USER = Collections.emptyList();
private static final List<VorgangStatus> COUNT_BY_VORGANG_EINHEITLICHER_ANSPRECHPARTNER = Collections.emptyList();
@@ -93,9 +95,7 @@ class StatisticService {
}
List<VorgangStatus> getVerwaltungUserStatuses() {
- return Stream
- .of(VorgangStatus.NEU, VorgangStatus.ANGENOMMEN, VorgangStatus.IN_BEARBEITUNG, VorgangStatus.BESCHIEDEN, VorgangStatus.ABGESCHLOSSEN,
- VorgangStatus.VERWORFEN, VorgangStatus.ZU_LOESCHEN, VorgangStatus.WEITERGELEITET)
+ return COUNT_BY_VORGANG_STATUS_VERWALTUNG_USER.stream()
.filter(status -> status != VorgangStatus.WEITERGELEITET || featureToggleProperties.isForwardByOzgCloudEnabled())
.toList();
}
diff --git a/alfa-service/src/test/java/de/ozgcloud/alfa/RootViewLinkHandlerTest.java b/alfa-service/src/test/java/de/ozgcloud/alfa/RootViewLinkHandlerTest.java
index be76fab90e..02d0843e8e 100644
--- a/alfa-service/src/test/java/de/ozgcloud/alfa/RootViewLinkHandlerTest.java
+++ b/alfa-service/src/test/java/de/ozgcloud/alfa/RootViewLinkHandlerTest.java
@@ -442,14 +442,14 @@ class RootViewLinkHandlerTest {
@BeforeEach
void mock() {
- doReturn(statuses.stream()).when(viewLinkHandler).getVorgangStatusStream();
+ doReturn(statuses.stream()).when(viewLinkHandler).getVorgangStatuses();
}
@Test
- void shouldCallGetVorgangStatusStream() {
+ void shouldCallGetVorgangStatuses() {
viewLinkHandler.buildVorgangStatusLinks(RootViewLinkHandler.VORGAENGE_ALL_REL_TEMPLATE, UserProfileTestFactory.ID);
- verify(viewLinkHandler).getVorgangStatusStream();
+ verify(viewLinkHandler).getVorgangStatuses();
}
@Test
@@ -773,15 +773,15 @@ class RootViewLinkHandlerTest {
@BeforeEach
void mock() {
- doReturn(vorgangStatuses.stream()).when(viewLinkHandler).getVorgangStatusStream();
+ doReturn(vorgangStatuses.stream()).when(viewLinkHandler).getVorgangStatuses();
doAnswer(AdditionalAnswers.returnsElementsOf(links)).when(viewLinkHandler).buildGetAllVorgaengeByStatus(any(), any());
}
@Test
- void shouldCallGetVorgangStatusStream() {
+ void shouldCallGetVorgangStatuses() {
viewLinkHandler.addVorgangStatusLinks(model, RootViewLinkHandler.VORGAENGE_ALL_REL_TEMPLATE);
- verify(viewLinkHandler).getVorgangStatusStream();
+ verify(viewLinkHandler).getVorgangStatuses();
}
@Test
@@ -801,11 +801,11 @@ class RootViewLinkHandlerTest {
}
@Nested
- class TestGetVorgangStatusStream {
+ class TestGetVorgangStatuses {
@Test
void shouldCallFeatureToggleProperties() {
- viewLinkHandler.getVorgangStatusStream().toList();
+ viewLinkHandler.getVorgangStatuses().toList();
verify(featureToggleProperties).isForwardByOzgCloudEnabled();
}
@@ -824,7 +824,7 @@ class RootViewLinkHandlerTest {
VorgangStatus.BESCHIEDEN, VorgangStatus.ABGESCHLOSSEN, VorgangStatus.VERWORFEN, VorgangStatus.ZU_LOESCHEN,
VorgangStatus.WEITERGELEITET);
- var stream = viewLinkHandler.getVorgangStatusStream();
+ var stream = viewLinkHandler.getVorgangStatuses();
assertThat(stream).containsExactlyInAnyOrderElementsOf(expectedStatuses);
}
@@ -843,7 +843,7 @@ class RootViewLinkHandlerTest {
var expectedStatuses = List.of(VorgangStatus.NEU, VorgangStatus.ANGENOMMEN, VorgangStatus.IN_BEARBEITUNG,
VorgangStatus.BESCHIEDEN, VorgangStatus.ABGESCHLOSSEN, VorgangStatus.VERWORFEN, VorgangStatus.ZU_LOESCHEN);
- var stream = viewLinkHandler.getVorgangStatusStream();
+ var stream = viewLinkHandler.getVorgangStatuses();
assertThat(stream).containsExactlyInAnyOrderElementsOf(expectedStatuses);
}
--
GitLab