Fix tests for Windows

Author: ethauvin

src/test/java/rife/bld/extension/junitreporter/JUnitXmlParserTest.java

@@ -19,6 +19,7 @@
 import org.junit.jupiter.api.DisplayName;
 import org.junit.jupiter.api.Nested;
 import org.junit.jupiter.api.Test;
+import org.junit.jupiter.api.condition.DisabledOnOs;
 import org.junit.jupiter.api.condition.EnabledOnOs;
 import org.junit.jupiter.api.condition.OS;
 import org.junit.jupiter.api.io.TempDir;
@@ -40,6 +41,7 @@
 import java.nio.file.Path;
 import java.util.Map;
 
+import static java.io.File.separatorChar;
 import static org.assertj.core.api.Assertions.*;
 import static org.mockito.ArgumentMatchers.any;
 import static rife.bld.extension.junitreporter.JUnitXmlParser.EOL;
@@ -278,7 +280,7 @@ void extractTestFailuresGroupedWithInvalidTimeFormats(@TempDir Path tempDir) thr
 
         @Test
         void extractTestFailuresGroupedWithInvalidXmlPath() {
-            var invalidPath = "nonexistent/path/to/file.xml";
+            var invalidPath = "nonexistent" + separatorChar + "file.xml";
 
             assertThatThrownBy(() -> JUnitXmlParser.extractTestFailuresGrouped(invalidPath))
                     .isInstanceOf(JUnitXmlParserException.class)
@@ -425,6 +427,7 @@ void extractTestFailuresGroupedWithNullTextContent(@TempDir Path tempDir) throws
         }
 
         @Test
+        @DisabledOnOs(OS.WINDOWS)
         void extractTestFailuresGroupedWithUnreadableFile(@TempDir Path tempDir) throws IOException {
             var tempFile = tempDir.resolve("test.xml");
             Files.createFile(tempFile);
@@ -1072,7 +1075,7 @@ void validateFileThrowsWhenFileDoesNotExist() {
         }
 
         @Test
-        @EnabledOnOs({OS.LINUX, OS.MAC})
+        @DisabledOnOs(OS.WINDOWS)
         void validateFileThrowsWhenFileIsNotReadable(@TempDir Path tempDir) throws IOException {
             var tempFile = tempDir.resolve("test.xml");
             Files.createFile(tempFile);

src/test/java/rife/bld/extension/junitreporter/ReportPrinterTests.java

@@ -178,22 +178,20 @@ void indentWithNullString(String input) {
     class PrintFailureTests {
         @Test
         void printFailureWithNullFailureIndex() {
-            var failure = new TestFailure("testMethod", "Test Method", "TestClass", "AssertionError",
-                    "Test failed message", "", 0.123);
+            var failure = new TestFailure("testMethod", "Test Method", "TestClass",
+                    "AssertionError", "Test failed message", "", 0.123);
 
             var outContent = new ByteArrayOutputStream();
             System.setOut(new PrintStream(outContent));
 
             ReportPrinter.printFailure(failure, 1, null);
 
-            var expectedOutput = """
-                    Test: testMethod
-                        - Name: Test Method
-                        - Type: AssertionError
-                        - Message:
-                            Test failed message
-                        - Time: 0.123
-                    """;
+            var expectedOutput = String.format("Test: testMethod%n" +
+                    "    - Name: Test Method%n" +
+                    "    - Type: AssertionError%n" +
+                    "    - Message:%n" +
+                    "        Test failed message%n" +
+                    "    - Time: 0.123%n");
 
             assertThat(outContent.toString().trim()).isEqualTo(expectedOutput.trim());
 
@@ -202,22 +200,20 @@ void printFailureWithNullFailureIndex() {
 
         @Test
         void printFailureWithNullIndices() {
-            var failure = new TestFailure("testMethod", "Test Method", "TestClass", "AssertionError",
-                    "Test failed message", "", 0.123);
+            var failure = new TestFailure("testMethod", "Test Method", "TestClass",
+                    "AssertionError", "Test failed message", "", 0.123);
 
             var outContent = new ByteArrayOutputStream();
             System.setOut(new PrintStream(outContent));
 
             ReportPrinter.printFailure(failure, null, null);
 
-            var expectedOutput = """
-                    Test: testMethod
-                        - Name: Test Method
-                        - Type: AssertionError
-                        - Message:
-                            Test failed message
-                        - Time: 0.123
-                    """;
+            var expectedOutput = String.format("Test: testMethod%n" +
+                    "    - Name: Test Method%n" +
+                    "    - Type: AssertionError%n" +
+                    "    - Message:%n" +
+                    "        Test failed message%n" +
+                    "    - Time: 0.123%n");
 
             assertThat(outContent.toString().trim()).isEqualTo(expectedOutput.trim());
 
@@ -226,22 +222,20 @@ void printFailureWithNullIndices() {
 
         @Test
         void printFailureWithValidIndices() {
-            var failure = new TestFailure("testMethod", "Test Method", "TestClass", "AssertionError",
-                    "Test failed message", "", 0.123);
+            var failure = new TestFailure("testMethod", "Test Method", "TestClass",
+                    "AssertionError", "Test failed message", "", 0.123);
 
             var outContent = new ByteArrayOutputStream();
             System.setOut(new PrintStream(outContent));
 
             ReportPrinter.printFailure(failure, 1, 2);
 
-            var expectedOutput = """
-                    [1.2] Test: testMethod
-                        - Name: Test Method
-                        - Type: AssertionError
-                        - Message:
-                            Test failed message
-                        - Time: 0.123
-                    """;
+            var expectedOutput = String.format("[1.2] Test: testMethod%n" +
+                    "    - Name: Test Method%n" +
+                    "    - Type: AssertionError%n" +
+                    "    - Message:%n" +
+                    "        Test failed message%n" +
+                    "    - Time: 0.123%n");
 
             assertThat(outContent.toString().trim()).isEqualTo(expectedOutput.trim());
 
@@ -325,10 +319,10 @@ void printSummaryWithEmptyGroupedFailures() {
         @Test
         @SuppressWarnings("ExtractMethodRecommender")
         void printSummaryWithNonEmptyGroupedFailures() {
-            var failure1 = new TestFailure("testMethod1", "Test Method 1", "TestClass1", "AssertionError",
-                    "Message: Test failed 1", "", 0.101);
-            var failure2 = new TestFailure("testMethod2", "", "TestClass1", "AssertionError",
-                    "Message: Test failed 2", "", 0.202);
+            var failure1 = new TestFailure("testMethod1", "Test Method 1", "TestClass1",
+                    "AssertionError", "Message: Test failed 1", "", 0.101);
+            var failure2 = new TestFailure("testMethod2", "", "TestClass1",
+                    "AssertionError", "Message: Test failed 2", "", 0.202);
 
             var failuresForClass1 = new TestClassFailures("TestClass1");
             failuresForClass1.addFailure(failure1);