From e770b81ea50b7be217ad5020612c843979bd736c Mon Sep 17 00:00:00 2001
From: Marcel <marcel@familienarchiv>
Date: Mon, 18 May 2026 14:58:34 +0200
Subject: [PATCH] fix(test): skip IOException test when running as root

setReadable(false) silently no-ops as root; check canRead() to guard
the assumption correctly so the test is skipped in Docker CI.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
---
 .../familienarchiv/importing/MassImportServiceTest.java        | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/backend/src/test/java/org/raddatz/familienarchiv/importing/MassImportServiceTest.java b/backend/src/test/java/org/raddatz/familienarchiv/importing/MassImportServiceTest.java
index 4442e15d..e6f81f57 100644
--- a/backend/src/test/java/org/raddatz/familienarchiv/importing/MassImportServiceTest.java
+++ b/backend/src/test/java/org/raddatz/familienarchiv/importing/MassImportServiceTest.java
@@ -575,7 +575,8 @@ class MassImportServiceTest {
         File unreadable = tempDir.resolve("unreadable.pdf").toFile();
         buildMinimalImportXlsx(tempDir, "unreadable.pdf");
         ReflectionTestUtils.setField(service, "importDir", tempDir.toString());
-        assumeTrue(unreadable.setReadable(false), "Requires non-root file permissions");
+        unreadable.setReadable(false);
+        assumeTrue(!unreadable.canRead(), "Requires non-root file permissions");
         try {
             service.runImportAsync();
             assertThat(service.getStatus().skipped()).isEqualTo(1);