From 1cae9ac31107a20aa7682896eb24b32fd42e4a5d Mon Sep 17 00:00:00 2001 From: Marcel Date: Mon, 18 May 2026 22:21:31 +0200 Subject: [PATCH] test(document): assert non-empty result in receiverSort lazy-loading test assertThatCode(() -> service.searchDocuments(...)) passed vacuously on an empty page; capture the result, assert totalElements > 0, then assert getSender().getLastName() is accessible post-return. Co-Authored-By: Claude Sonnet 4.6 --- .../familienarchiv/document/DocumentLazyLoadingTest.java | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/backend/src/test/java/org/raddatz/familienarchiv/document/DocumentLazyLoadingTest.java b/backend/src/test/java/org/raddatz/familienarchiv/document/DocumentLazyLoadingTest.java index 5123337f..2762df64 100644 --- a/backend/src/test/java/org/raddatz/familienarchiv/document/DocumentLazyLoadingTest.java +++ b/backend/src/test/java/org/raddatz/familienarchiv/document/DocumentLazyLoadingTest.java @@ -119,10 +119,13 @@ class DocumentLazyLoadingTest { .tags(new HashSet<>(Set.of(tag))) .build()); - assertThatCode(() -> documentService.searchDocuments( + DocumentSearchResult result = documentService.searchDocuments( null, null, null, null, null, null, null, null, DocumentSort.RECEIVER, "asc", null, - PageRequest.of(0, 20))) + PageRequest.of(0, 20)); + assertThat(result.totalElements()).isGreaterThan(0); + assertThatCode(() -> + result.items().forEach(i -> i.document().getSender().getLastName())) .doesNotThrowAnyException(); }