Application configuration directory not required to be writable

.gitea/workflows/build.yaml

@@ -4,12 +4,10 @@ on:
     branches: [ master ]
 jobs:
   build:
-    runs-on: hostinger
+    runs-on: woryzen
     steps:
       - name: Checkout sources
         uses: actions/checkout@v4
-      - name: Setup Gradle
-        uses: gradle/actions/setup-gradle@v4
       - name: Execute Gradle build
         env:
           PUBLISHER_TOKEN: ${{ secrets.PUBLISHER_TOKEN }}

gradle.properties

@@ -3,6 +3,6 @@ org.gradle.parallel=true
 org.gradle.caching=true
 
 gitea.maven.url = https://gitea.woggioni.net/api/packages/woggioni/maven
-jwo.version = 2025.04.19
+jwo.version = 2025.06.10
-lys.version = 2025.04.16
+lys.version = 2025.06.03
 guice.version = 5.0.1

src/main/java/net/woggioni/jwo/Application.java

@@ -35,7 +35,7 @@ public class Application {
         return new Builder().name(name);
     }
 
-    private static boolean validateWritableDirectory(Path candidate) {
+    private static boolean validateDirectory(Path candidate, boolean writable) {
         try {
             if (!Files.exists(candidate)) {
                 Files.createDirectories(candidate);
@@ -44,7 +44,7 @@ public class Application {
             } else if (!Files.isDirectory(candidate)) {
                 log.trace("Directory '{}' discarded because it is not a directory", candidate);
                 return false;
-            } else if (!Files.isWritable(candidate)) {
+            } else if (writable && !Files.isWritable(candidate)) {
                 log.trace("Directory '{}' discarded because it is not writable", candidate);
                 return false;
             } else {
@@ -61,9 +61,9 @@ public class Application {
     }
 
     @SneakyThrows
-    private Path selectWritableDirectory(Stream<Path> candidates, String successMessage, String errorMessage) {
+    private Path selectDirectory(Stream<Path> candidates, boolean writable, String successMessage, String errorMessage) {
         return candidates
-                .filter(Application::validateWritableDirectory)
+                .filter(p -> validateDirectory(p, writable))
                 .peek(p -> log.debug(successMessage, p))
                 .findFirst()
                 .orElseThrow((Sup<Throwable>) () -> new FileNotFoundException(errorMessage));
@@ -99,7 +99,8 @@ public class Application {
                     Optional.ofNullable(System.getProperty("user.home"))
                             .map(prefix -> Paths.get(prefix, "." + name, "cache")));
         }
-        return selectWritableDirectory(streamCat(commonCandidates, osSpecificCandidates),
+        return selectDirectory(streamCat(commonCandidates, osSpecificCandidates),
+                true,
                 "Using cache directory '{}'",
                 "Unable to find a usable cache directory");
     }
@@ -134,13 +135,14 @@ public class Application {
                     Optional.ofNullable(System.getProperty("user.home"))
                             .map(prefix -> Paths.get(prefix, "." + name)));
         }
-        return selectWritableDirectory(streamCat(commonCandidates, osSpecificCandidates),
+        return selectDirectory(streamCat(commonCandidates, osSpecificCandidates),
+                true,
                 "Using data directory '{}'",
                 "Unable to find a usable data directory");
     }
 
     @SneakyThrows
-    public Path computeConfigurationDirectory() {
+    public Path computeConfigurationDirectory(boolean writable) {
         Stream<Path> commonCandidates = optional2Stream(
                 Optional.ofNullable(configurationDirectoryPropertyKey).map(System::getProperty).map(Paths::get),
                 Optional.ofNullable(configurationDirectoryEnvVar).map(System::getenv).map(Paths::get)
@@ -169,7 +171,8 @@ public class Application {
                     Optional.ofNullable(System.getProperty("user.home")
                     ).map(prefix -> Paths.get(prefix, "." + name, "config")));
         }
-        return selectWritableDirectory(streamCat(commonCandidates, osSpecificCandidates),
+        return selectDirectory(streamCat(commonCandidates, osSpecificCandidates),
+                writable,
                 "Using configuration directory '{}'",
                 "Unable to find a usable configuration directory");
     }

src/test/java/net/woggioni/jwo/ApplicationTest.java

@@ -14,7 +14,7 @@ public class ApplicationTest {
             .dataDirectoryEnvVar("APP_DATA_DIR")
             .configurationDirectoryEnvVar("APP_CONF_DIR")
             .build();
-        final var confDir = app.computeConfigurationDirectory();
+        final var confDir = app.computeConfigurationDirectory(true);
         final var cacheDir = app.computeCacheDirectory();
         final var dataDir = app.computeDataDirectory();
     }