Window Test Support

buildSrc/src/main/kotlin/core-config.gradle.kts

@@ -31,7 +31,7 @@ dependencies {
     testFixturesApi("org.mockito:mockito-inline:4.7.0")
     testFixturesApi("org.mockito:mockito-junit-jupiter:4.7.0")
     testFixturesApi("org.hamcrest:hamcrest-all:1.3")
-    testFixturesApi("com.google.jimfs:jimfs:1.2")
+    testFixturesApi("com.google.jimfs:jimfs:1.3.0")
 }
 
 publishing {

configlib-core/src/test/java/de/exlll/configlib/ExampleConfigurationTests.java

@@ -30,7 +30,7 @@ class ExampleConfigurationTests {
             .build();
 
     private final FileSystem fs = Jimfs.newFileSystem();
-    private final Path yamlFile = fs.getPath("/tmp/config.yml");
+    private final Path yamlFile = fs.getPath(TestUtils.createPlatformSpecificFilePath("/tmp/config.yml"));
 
     @BeforeEach
     void setUp() throws IOException {

configlib-core/src/test/java/de/exlll/configlib/SerializersTest.java

@@ -29,6 +29,15 @@
 import static org.hamcrest.Matchers.*;
 
 class SerializersTest {
+
+    private final String tmpConfigPath = TestUtils.createPlatformSpecificFilePath("/tmp/config.yml");
+    /*
+        If found a way to implement the newline char test on the file name without
+         giving errors on windows. The following field ought to be changed to do so.
+     */
+    private final String tmpWithUnderscorePath = TestUtils.createPlatformSpecificFilePath("/tmp/with_underscore.yml");
+    private final String tmpPath = TestUtils.createPlatformSpecificFilePath("/tmp");
+
     @Test
     void booleanSerializer() {
         Serializer<Boolean, Boolean> serializer = new Serializers.BooleanSerializer();
@@ -461,42 +470,34 @@ void uuidSerializer() {
     void fileSerializer() {
         Serializer<File, String> serializer = new Serializers.FileSerializer();
 
-        String path1 = "/tmp/config.yml";
-        String path2 = "/tmp/with \n new \n lines.yml";
-        String path3 = "/tmp";
+        File file1 = new File(tmpConfigPath);
+        File file2 = new File(tmpWithUnderscorePath);
+        File file3 = new File(tmpPath);
 
-        File file1 = new File(path1);
-        File file2 = new File(path2);
-        File file3 = new File(path3);
+        assertThat(serializer.serialize(file1), is(tmpConfigPath));
+        assertThat(serializer.serialize(file2), is(tmpWithUnderscorePath));
+        assertThat(serializer.serialize(file3), is(tmpPath));
 
-        assertThat(serializer.serialize(file1), is(path1));
-        assertThat(serializer.serialize(file2), is(path2));
-        assertThat(serializer.serialize(file3), is(path3));
-
-        assertThat(serializer.deserialize(path1), is(file1));
-        assertThat(serializer.deserialize(path2), is(file2));
-        assertThat(serializer.deserialize(path3), is(file3));
+        assertThat(serializer.deserialize(tmpConfigPath), is(file1));
+        assertThat(serializer.deserialize(tmpWithUnderscorePath), is(file2));
+        assertThat(serializer.deserialize(tmpPath), is(file3));
     }
 
     @Test
     void pathSerializer() {
         Serializer<Path, String> serializer = new Serializers.PathSerializer();
 
-        String path1 = "/tmp/config.yml";
-        String path2 = "/tmp/with \n new \n lines.yml";
-        String path3 = "/tmp";
-
-        Path file1 = Path.of(path1);
-        Path file2 = Path.of(path2);
-        Path file3 = Path.of(path3);
+        Path file1 = Path.of(tmpConfigPath);
+        Path file2 = Path.of(tmpWithUnderscorePath);
+        Path file3 = Path.of(tmpPath);
 
-        assertThat(serializer.serialize(file1), is(path1));
-        assertThat(serializer.serialize(file2), is(path2));
-        assertThat(serializer.serialize(file3), is(path3));
+        assertThat(serializer.serialize(file1), is(tmpConfigPath));
+        assertThat(serializer.serialize(file2), is(tmpWithUnderscorePath));
+        assertThat(serializer.serialize(file3), is(tmpPath));
 
-        assertThat(serializer.deserialize(path1), is(file1));
-        assertThat(serializer.deserialize(path2), is(file2));
-        assertThat(serializer.deserialize(path3), is(file3));
+        assertThat(serializer.deserialize(tmpConfigPath), is(file1));
+        assertThat(serializer.deserialize(tmpWithUnderscorePath), is(file2));
+        assertThat(serializer.deserialize(tmpPath), is(file3));
     }
 
     @Test

configlib-core/src/testFixtures/java/de/exlll/configlib/TestUtils.java

@@ -4,6 +4,7 @@
 import org.junit.jupiter.api.function.Executable;
 
 import java.awt.Point;
+import java.io.File;
 import java.io.IOException;
 import java.lang.reflect.Field;
 import java.math.BigInteger;
@@ -289,4 +290,26 @@ public static ConfigurationElement<?> fieldAsElement(Class<?> type, String field
         Field field = getField(type, fieldName);
         return new ConfigurationElements.FieldElement(field);
     }
-}
+
+    /*
+        There were absolute path errors when trying to pass the unit tests
+        on different platforms like Windows. Currently, Jimfs(1.3.0) lacks support
+        for both absolutes paths and relative paths on Windows, see:
+        - https://github.com/google/jimfs/issues/69
+        - https://github.com/google/jimfs/blob/master/jimfs/src/main/java/com/google/common/jimfs/WindowsPathType.java
+
+        So, in order to run unit tests on Windows. We have to translate the current
+        path declarations to fulfill the non-unix system's needs.
+    */
+    public static String createPlatformSpecificFilePath(String path) {
+        final String platform = System.getProperty("os.name");
+
+        if (!platform.contains("Windows")) return path;
+
+        return String.format("C:%s", path.replace("/", File.separator));
+    }
+
+    public static List<String> createListOfPlatformSpecificFilePaths(String... paths) {
+        return Stream.of(paths).map(TestUtils::createPlatformSpecificFilePath).toList();
+    }
+}

configlib-core/src/testFixtures/java/de/exlll/configlib/configurations/ExampleConfigurationsSerialized.java

@@ -121,8 +121,8 @@ public final class ExampleConfigurationsSerialized {
             entry("a1_localDateTime", "2000-01-01T00:00"),
             entry("a1_instant", "0000-01-01T00:00:00Z"),
             entry("a1_uuid", "d50f3bdd-ac66-4b74-a01f-4617b24d68c0"),
-            entry("a1_file", "/tmp"),
-            entry("a1_path", "/tmp"),
+            entry("a1_file", TestUtils.createPlatformSpecificFilePath("/tmp")),
+            entry("a1_path", TestUtils.createPlatformSpecificFilePath("/tmp")),
             entry("a1_url", "https://example.com"),
             entry("a1_uri", "https://example.com"),
             entry("a1_Enm", "A"),
@@ -146,8 +146,8 @@ public final class ExampleConfigurationsSerialized {
             entry("a1_listLocalDateTime", List.of("2000-01-01T00:00", "2000-01-02T00:00", "2000-01-03T00:00")),
             entry("a1_listInstant", List.of("0000-01-01T00:00:00Z", "0001-01-01T00:00:00Z", "-1000000000-01-01T00:00:00Z")),
             entry("a1_listUuid", List.of("d50f3bdd-ac66-4b74-a01f-4617b24d68c0", "d50f3bdd-ac66-4b74-a01f-4617b24d68c1", "d50f3bdd-ac66-4b74-a01f-4617b24d68c2")),
-            entry("a1_listFile", List.of("/tmp", "/tmp/config.yml", "/tmp/with \n new \n lines.yml")),
-            entry("a1_listPath", List.of("/tmp", "/tmp/config.yml", "/tmp/with \n new \n lines.yml")),
+            entry("a1_listFile", TestUtils.createListOfPlatformSpecificFilePaths("/tmp", "/tmp/config.yml", "/tmp/with_underscore.yml")),
+            entry("a1_listPath", TestUtils.createListOfPlatformSpecificFilePaths("/tmp", "/tmp/config.yml", "/tmp/with_underscore.yml")),
             entry("a1_listUrl", List.of("https://example.com", "https://example.com?query=yes", "https://example.com?query=yes#fragment=true")),
             entry("a1_listUri", List.of("https://example.com", "https://example.com?query=yes", "https://example.com?query=yes#fragment=true")),
             entry("a1_listEnm", List.of("A", "B", "C")),
@@ -299,8 +299,8 @@ public final class ExampleConfigurationsSerialized {
             entry("a2_localDateTime", "2000-01-02T00:00"),
             entry("a2_instant", "0001-01-01T00:00:00Z"),
             entry("a2_uuid", "d50f3bdd-ac66-4b74-a01f-4617b24d68c1"),
-            entry("a2_file", "/tmp/config.yml"),
-            entry("a2_path", "/tmp/config.yml"),
+            entry("a2_file", TestUtils.createPlatformSpecificFilePath("/tmp/config.yml")),
+            entry("a2_path", TestUtils.createPlatformSpecificFilePath("/tmp/config.yml")),
             entry("a2_url", "https://example.com?query=yes"),
             entry("a2_uri", "https://example.com?query=yes"),
             entry("a2_Enm", "B"),
@@ -324,8 +324,8 @@ public final class ExampleConfigurationsSerialized {
             entry("a2_listLocalDateTime", List.of("2000-01-02T00:00", "2000-01-03T00:00", "2000-01-04T00:00")),
             entry("a2_listInstant", List.of("0001-01-01T00:00:00Z", "-1000000000-01-01T00:00:00Z", "+1000000000-12-31T23:59:59.999999999Z")),
             entry("a2_listUuid", List.of("d50f3bdd-ac66-4b74-a01f-4617b24d68c1", "d50f3bdd-ac66-4b74-a01f-4617b24d68c2", "d50f3bdd-ac66-4b74-a01f-4617b24d68c3")),
-            entry("a2_listFile", List.of("/tmp/config.yml", "/tmp/with \n new \n lines.yml", "with \n new \n lines.yml")),
-            entry("a2_listPath", List.of("/tmp/config.yml", "/tmp/with \n new \n lines.yml", "with \n new \n lines.yml")),
+            entry("a2_listFile", TestUtils.createListOfPlatformSpecificFilePaths("/tmp/config.yml", "/tmp/with_underscore.yml", "with_underscore.yml")),
+            entry("a2_listPath", TestUtils.createListOfPlatformSpecificFilePaths("/tmp/config.yml", "/tmp/with_underscore.yml", "with_underscore.yml")),
             entry("a2_listUrl", List.of("https://example.com?query=yes", "https://example.com?query=yes#fragment=true", "https://example.com#fragment=true")),
             entry("a2_listUri", List.of("https://example.com?query=yes", "https://example.com?query=yes#fragment=true", "https://example.com#fragment=true")),
             entry("a2_listEnm", List.of("B", "C", "D")),

configlib-core/src/testFixtures/java/de/exlll/configlib/configurations/ExampleInitializer.java

@@ -1,5 +1,7 @@
 package de.exlll.configlib.configurations;
 
+import de.exlll.configlib.TestUtils;
+
 import java.awt.Point;
 import java.io.File;
 import java.math.BigDecimal;
@@ -59,15 +61,15 @@ public final class ExampleInitializer {
     private static final UUID UUID_4 = UUID.fromString("d50f3bdd-ac66-4b74-a01f-4617b24d68c3");
     private static final UUID UUID_5 = UUID.fromString("d50f3bdd-ac66-4b74-a01f-4617b24d68c4");
 
-    private static final File FILE_1 = new File("/tmp");
-    private static final File FILE_2 = new File("/tmp/config.yml");
-    private static final File FILE_3 = new File("/tmp/with \n new \n lines.yml");
-    private static final File FILE_4 = new File("with \n new \n lines.yml");
+    private static final File FILE_1 = new File(TestUtils.createPlatformSpecificFilePath("/tmp"));
+    private static final File FILE_2 = new File(TestUtils.createPlatformSpecificFilePath("/tmp/config.yml"));
+    private static final File FILE_3 = new File(TestUtils.createPlatformSpecificFilePath("/tmp/with_underscore.yml"));
+    private static final File FILE_4 = new File(TestUtils.createPlatformSpecificFilePath("with_underscore.yml"));
 
-    private static final Path PATH_1 = Path.of("/tmp");
-    private static final Path PATH_2 = Path.of("/tmp/config.yml");
-    private static final Path PATH_3 = Path.of("/tmp/with \n new \n lines.yml");
-    private static final Path PATH_4 = Path.of("with \n new \n lines.yml");
+    private static final Path PATH_1 = Path.of(TestUtils.createPlatformSpecificFilePath("/tmp"));
+    private static final Path PATH_2 = Path.of(TestUtils.createPlatformSpecificFilePath("/tmp/config.yml"));
+    private static final Path PATH_3 = Path.of(TestUtils.createPlatformSpecificFilePath("/tmp/with_underscore.yml"));
+    private static final Path PATH_4 = Path.of(TestUtils.createPlatformSpecificFilePath("with_underscore.yml"));
 
     private static final URL URL_1 = createUrl("https://example.com");
     private static final URL URL_2 = createUrl("https://example.com?query=yes");

configlib-yaml/src/test/java/de/exlll/configlib/ExampleConfigurationYamlTests.java

@@ -19,7 +19,7 @@
 
 final class ExampleConfigurationYamlTests {
     private final FileSystem fs = Jimfs.newFileSystem();
-    private final Path yamlFile = fs.getPath("/tmp/config.yml");
+    private final Path yamlFile = fs.getPath(TestUtils.createPlatformSpecificFilePath("/tmp/config.yml"));
 
     @BeforeEach
     void setUp() throws IOException {

configlib-yaml/src/test/java/de/exlll/configlib/YamlConfigurationStoreTest.java

@@ -16,7 +16,11 @@
 
 class YamlConfigurationStoreTest {
     private final FileSystem fs = Jimfs.newFileSystem();
-    private final Path yamlFile = fs.getPath("/tmp/config.yml");
+
+    private final String yamlFilePath = TestUtils.createPlatformSpecificFilePath("/tmp/config.yml");
+    private final Path yamlFile = fs.getPath(yamlFilePath);
+
+    private final String abcFilePath = TestUtils.createPlatformSpecificFilePath("/a/b/c.yml");
 
     @BeforeEach
     void setUp() throws IOException {
@@ -208,7 +212,7 @@ void loadInvalidYaml() throws IOException {
 
         assertThrowsConfigurationException(
                 () -> store.load(yamlFile),
-                "The configuration file at /tmp/config.yml does not contain valid YAML."
+                String.format("The configuration file at %s does not contain valid YAML.", yamlFilePath)
         );
     }
 
@@ -220,7 +224,7 @@ void loadEmptyYaml() throws IOException {
 
         assertThrowsConfigurationException(
                 () -> store.load(yamlFile),
-                "The configuration file at /tmp/config.yml is empty or only contains null."
+            String.format("The configuration file at %s is empty or only contains null.", yamlFilePath)
         );
     }
 
@@ -232,9 +236,9 @@ void loadNonMapYaml() throws IOException {
 
         assertThrowsConfigurationException(
                 () -> store.load(yamlFile),
-                "The contents of the YAML file at /tmp/config.yml do not represent a " +
+                String.format("The contents of the YAML file at %s do not represent a " +
                 "configuration. A valid configuration file contains a YAML map but instead a " +
-                "'class java.lang.String' was found."
+                "'class java.lang.String' was found.", yamlFilePath)
         );
     }
 
@@ -261,7 +265,7 @@ void saveConfigurationWithInvalidTargetType() {
     void saveCreatesParentDirectoriesIfPropertyTrue() {
         YamlConfigurationStore<A> store = newDefaultStore(A.class);
 
-        Path file = fs.getPath("/a/b/c.yml");
+        Path file = fs.getPath(abcFilePath);
         store.save(new A(), file);
 
         assertTrue(Files.exists(file.getParent()));
@@ -275,10 +279,10 @@ void saveDoesNotCreateParentDirectoriesIfPropertyFalse() {
                 .build();
         YamlConfigurationStore<A> store = new YamlConfigurationStore<>(A.class, properties);
 
-        Path file = fs.getPath("/a/b/c.yml");
+        Path file = fs.getPath(abcFilePath);
         assertThrowsRuntimeException(
                 () -> store.save(new A(), file),
-                "java.nio.file.NoSuchFileException: /a/b/c.yml"
+                String.format("java.nio.file.NoSuchFileException: %s", abcFilePath)
         );
     }
 

configlib-yaml/src/test/java/de/exlll/configlib/YamlConfigurationsTest.java

@@ -15,7 +15,7 @@
 class YamlConfigurationsTest {
     private static final FieldFilter includeI = field -> field.getName().equals("i");
     private final FileSystem fs = Jimfs.newFileSystem();
-    private final Path yamlFile = fs.getPath("/tmp/config.yml");
+    private final Path yamlFile = fs.getPath(TestUtils.createPlatformSpecificFilePath("/tmp/config.yml"));
 
     @BeforeEach
     void setUp() throws IOException {

configlib-yaml/src/test/java/de/exlll/configlib/YamlFileWriterTest.java

@@ -21,7 +21,7 @@
 @SuppressWarnings("unused")
 class YamlFileWriterTest {
     private final FileSystem fs = Jimfs.newFileSystem();
-    private final Path yamlFile = fs.getPath("/tmp/config.yml");
+    private final Path yamlFile = fs.getPath(TestUtils.createPlatformSpecificFilePath("/tmp/config.yml"));
 
     @BeforeEach
     void setUp() throws IOException {