ci: fix: handle missing docker config on MacOS & Windows in ci-test.yml

Author: Abega1642

.github/workflows/ci-test.yml

@@ -9,7 +9,7 @@ on:
 jobs:
   test:
     runs-on: ${{ matrix.os }}
-    timeout-minutes: 30
+    timeout-minutes: 45
     permissions:
       contents: read
     strategy:
@@ -22,6 +22,34 @@ jobs:
       - name: Checkout code
         uses: actions/checkout@v6.0.1
 
+      - name: Set up Docker (Windows)
+        if: runner.os == 'Windows'
+        shell: bash
+        run: |
+          echo "Checking Docker status on Windows..."
+          # Windows runners have Docker pre-installed and running
+          docker version
+          docker info
+          echo "Docker is available"
+
+      - name: Set up Docker (macOS)
+        if: runner.os == 'macOS'
+        run: |
+          brew install colima docker docker-compose
+          # Use QEMU instead of VZ for better compatibility in CI
+          colima start --cpu 2 --memory 4 --vm-type=qemu
+          echo "Waiting for Docker to be ready..."
+          for i in {1..30}; do
+            if docker info > /dev/null 2>&1; then
+              echo "Docker is ready"
+              break
+            fi
+            echo "Waiting for Docker... ($i/30)"
+            sleep 10
+          done
+          docker version
+          docker info
+
       - name: Set up JDK ${{ matrix.java-version }}
         uses: actions/setup-java@v5
         with:
@@ -43,6 +71,27 @@ jobs:
           restore-keys: |
             ${{ runner.os }}-gradle-
 
+      - name: Configure Docker environment (Windows)
+        if: runner.os == 'Windows'
+        shell: bash
+        run: |
+          echo "DOCKER_HOST=npipe:////./pipe/docker_engine" >> $GITHUB_ENV
+          echo "TESTCONTAINERS_RYUK_DISABLED=true" >> $GITHUB_ENV
+          echo "TESTCONTAINERS_CHECKS_DISABLE=true" >> $GITHUB_ENV
+
+      - name: Verify Docker is running (Unix)
+        if: runner.os != 'Windows'
+        run: |
+          docker version
+          docker info
+
+      - name: Verify Docker is running (Windows)
+        if: runner.os == 'Windows'
+        shell: bash
+        run: |
+          docker version
+          docker info
+
       - name: Run tests (Unix)
         if: runner.os != 'Windows'
         run: make ci-test
@@ -56,7 +105,7 @@ jobs:
         if: always()
         uses: actions/upload-artifact@v4
         with:
-          name: test-results-${{ matrix.os }}
+          name: test-results-${{ matrix.os }}-java${{ matrix.java-version }}
           path: |
             build/reports/tests/
             build/test-results/

src/test/java/com/example/arinfra/file/SecureTempFileManagerTest.java

@@ -308,12 +308,29 @@ void should_create_writable_file() throws IOException {
   }
 
   @Test
-  void should_not_create_executable_file() throws IOException {
+  void should_not_create_executable_file_on_unix() throws IOException {
     createdFile = subject.createSecureTempFile(TEST_PREFIX, TEST_SUFFIX);
 
     assertFalse(createdFile.canExecute());
   }
 
+  @Test
+  @EnabledOnOs(OS.WINDOWS)
+  void should_handle_windows_executable_behavior() throws IOException {
+    createdFile = subject.createSecureTempFile(TEST_PREFIX, TEST_SUFFIX);
+
+    // Windows doesn't use Unix-style executable bits
+    // Files are executable based on extension (.exe, .bat, .cmd, etc.)
+    // A .txt file should not be executable on Windows
+    assertTrue(createdFile.isFile());
+    assertFalse(createdFile.isDirectory());
+
+    String userTempDir = System.getProperty("java.io.tmpdir");
+    assertTrue(createdFile.getAbsolutePath().startsWith(userTempDir));
+
+    // On Windows, canExecute() may return true for all files the user owns
+  }
+
   @Test
   void should_create_temp_directory_with_prefix() throws IOException {
     File createdDir = subject.createSecureTempDirectory(TEST_PREFIX);