chain assertions

urlaubsverwaltung · May 17, 2023 · 7a1a712 · 7a1a712
1 parent 2c2a315
commit 7a1a712
Showing 1 changed file with 29 additions and 26 deletions.
diff --git a/src/test/java/de/focusshift/zeiterfassung/timeentry/TimeEntryServiceImplTest.java b/src/test/java/de/focusshift/zeiterfassung/timeentry/TimeEntryServiceImplTest.java
@@ -788,17 +788,18 @@ void ensureGetEntriesForAllUsers() {
         final ZonedDateTime expectedBreakStart = ZonedDateTime.of(entryBreakStart, ZONE_ID_UTC);
         final ZonedDateTime expectedBreakEnd = ZonedDateTime.of(entryBreakEnd, ZONE_ID_UTC);
 
-        assertThat(actual).hasSize(2);
-        assertThat(actual).hasEntrySatisfying(batmanLocalId, timeEntries -> {
-            assertThat(timeEntries).containsExactly(
-                new TimeEntry(new TimeEntryId(1L), new UserId("batman"), "hard work", expectedStart, expectedEnd, false)
-            );
-        });
-        assertThat(actual).hasEntrySatisfying(pinguinLocalId, timeEntries -> {
-            assertThat(timeEntries).containsExactly(
-                new TimeEntry(new TimeEntryId(2L), new UserId("pinguin"), "deserved break", expectedBreakStart, expectedBreakEnd, true)
-            );
-        });
+        assertThat(actual)
+            .hasSize(2)
+            .hasEntrySatisfying(batmanLocalId, timeEntries -> {
+                assertThat(timeEntries).containsExactly(
+                    new TimeEntry(new TimeEntryId(1L), new UserId("batman"), "hard work", expectedStart, expectedEnd, false)
+                );
+            })
+            .hasEntrySatisfying(pinguinLocalId, timeEntries -> {
+                assertThat(timeEntries).containsExactly(
+                    new TimeEntry(new TimeEntryId(2L), new UserId("pinguin"), "deserved break", expectedBreakStart, expectedBreakEnd, true)
+                );
+            });
     }
 
     @Test
@@ -834,17 +835,18 @@ void ensureGetEntriesByUserLocalIds() {
         final ZonedDateTime expectedBreakStart = ZonedDateTime.of(entryBreakStart, ZONE_ID_UTC);
         final ZonedDateTime expectedBreakEnd = ZonedDateTime.of(entryBreakEnd, ZONE_ID_UTC);
 
-        assertThat(actual).hasSize(2);
-        assertThat(actual).hasEntrySatisfying(batmanLocalId, timeEntries -> {
-            assertThat(timeEntries).containsExactly(
-                new TimeEntry(new TimeEntryId(1L), new UserId("uuid-1"), "hard work", expectedStart, expectedEnd, false)
-            );
-        });
-        assertThat(actual).hasEntrySatisfying(robinLocalId, timeEntries -> {
-            assertThat(timeEntries).containsExactly(
-                new TimeEntry(new TimeEntryId(2L), new UserId("uuid-2"), "deserved break", expectedBreakStart, expectedBreakEnd, true)
-            );
-        });
+        assertThat(actual)
+            .hasSize(2)
+            .hasEntrySatisfying(batmanLocalId, timeEntries -> {
+                assertThat(timeEntries).containsExactly(
+                    new TimeEntry(new TimeEntryId(1L), new UserId("uuid-1"), "hard work", expectedStart, expectedEnd, false)
+                );
+            })
+            .hasEntrySatisfying(robinLocalId, timeEntries -> {
+                assertThat(timeEntries).containsExactly(
+                    new TimeEntry(new TimeEntryId(2L), new UserId("uuid-2"), "deserved break", expectedBreakStart, expectedBreakEnd, true)
+                );
+            });
     }
 
     @Test
@@ -862,10 +864,11 @@ void ensureGetEntriesByUserLocalIdsReturnsValuesForEveryAskedUserLocalId() {
 
         final Map<UserLocalId, List<TimeEntry>> actual = sut.getEntriesByUserLocalIds(from, toExclusive, List.of(batmanLocalId));
 
-        assertThat(actual).hasSize(1);
-        assertThat(actual).hasEntrySatisfying(batmanLocalId, timeEntries -> {
-            assertThat(timeEntries).isEmpty();
-        });
+        assertThat(actual)
+            .hasSize(1)
+            .hasEntrySatisfying(batmanLocalId, timeEntries -> {
+                assertThat(timeEntries).isEmpty();
+            });
     }
 
     @Test