add unit test

it-at-m · Feb 26, 2025 · 8808775 · 8808775
1 parent ecd8434
commit 8808775
Show file tree

Hide file tree

Showing 2 changed files with 27 additions and 4 deletions.
diff --git a/src/main/java/de/muenchen/dave/services/messstelle/UnauffaelligeTageReceiver.java b/src/main/java/de/muenchen/dave/services/messstelle/UnauffaelligeTageReceiver.java
@@ -72,15 +72,16 @@ public void loadMessstellenCron() {
      *             unauffälligen Tag gefunden wurde.
      */
     protected List<UnauffaelligerTag> loadUnauffaelligeTageForEachMessstelle() {
-        final var unaufaelligerTag = unauffaelligeTageRepository.findTopByOrderByKalendertagDatumDesc();
-        final var lastUnauffaelligerTag = unaufaelligerTag
+        final var lastUnaufaelligerTag = unauffaelligeTageRepository.findTopByOrderByKalendertagDatumDesc();
+        final var dayAfterLastUnauffaelligerTag = lastUnaufaelligerTag
                 .map(unauffaelligerTag -> unauffaelligerTag.getKalendertag().getDatum().plusDays(1))
                 .orElse(EARLIEST_DAY);
         final var yesterday = LocalDate.now().minusDays(1);
         final List<UnauffaelligerTagDto> unauffaelligeTage;
-        if (yesterday.isAfter(lastUnauffaelligerTag)) {
+        if (!yesterday.isBefore(dayAfterLastUnauffaelligerTag)) {
             unauffaelligeTage = Objects
-                    .requireNonNull(messstelleApi.getUnauffaelligeTageForEachMessstelleWithHttpInfo(lastUnauffaelligerTag, yesterday).block().getBody());
+                    .requireNonNull(
+                            messstelleApi.getUnauffaelligeTageForEachMessstelleWithHttpInfo(dayAfterLastUnauffaelligerTag, yesterday).block().getBody());
         } else {
             unauffaelligeTage = List.of();
         }

diff --git a/src/test/java/de/muenchen/dave/services/messstelle/UnauffaelligeTageReceiverTest.java b/src/test/java/de/muenchen/dave/services/messstelle/UnauffaelligeTageReceiverTest.java
@@ -266,6 +266,28 @@ void loadUnauffaelligeTageForEachMessstelleNoKaledertagForDatumFound() {
                 .findByDatum(LocalDate.of(2025, 2, 3));
     }
 
+    @Test
+    void loadUnauffaelligeTageForEachMessstelleYesterdayIsNotAfterLastUnauffaelligerTag() {
+        final var kalenderTagForYoungestSavedUnauffaelligerTag = new Kalendertag();
+        kalenderTagForYoungestSavedUnauffaelligerTag.setDatum(LocalDate.now().minusDays(1));
+
+        final var youngestSavedUnauffaelligerTag = new UnauffaelligerTag();
+        youngestSavedUnauffaelligerTag.setKalendertag(kalenderTagForYoungestSavedUnauffaelligerTag);
+        youngestSavedUnauffaelligerTag.setMstId(1234);
+
+        Mockito.when(unauffaelligeTageRepository.findTopByOrderByKalendertagDatumDesc()).thenReturn(Optional.of(youngestSavedUnauffaelligerTag));
+
+        final var result = unauffaelligeTageReceiver.loadUnauffaelligeTageForEachMessstelle();
+
+        Assertions.assertEquals(List.of(), result);
+
+        Mockito.verify(unauffaelligeTageRepository, Mockito.times(1))
+                .findTopByOrderByKalendertagDatumDesc();
+
+        Mockito.verify(messstelleApi, Mockito.times(0))
+                .getUnauffaelligeTageForEachMessstelleWithHttpInfo(Mockito.any(), Mockito.any());
+    }
+
     @Test
     void mapDto2Entity() {
         final var unauffaelligerTagDto = new UnauffaelligerTagDto();