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 99d9c9dffd8948c4cfaafd1ecbe678fcdc6318ed..6b1eddb06bbdde10b0b234ace883e20d2978bc65 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 9710c034641eca7da5f9085ff0b6ef0a3b832ac6..57b9c2e7c56f6c53b3027d44972243c109c7ab39 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 be76fab90ec9e76c219590ab474c9c017d455132..02d0843e8e8ebdd8673b51a9205435e1c09e1ca4 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); }