OZG-7572 apply code review comments

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) {

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();
 	}

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);
 			}