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,52 @@ jobs:
       - name: Checkout code
         uses: actions/checkout@v6.0.1
 
+      - name: Set up Docker (Windows)
+        if: runner.os == 'Windows'
+        shell: powershell
+        run: |
+          Write-Host "Starting Docker Desktop..."
+          Start-Process "C:\Program Files\Docker\Docker\Docker Desktop.exe" -ErrorAction SilentlyContinue
+          Write-Host "Waiting for Docker to be ready..."
+          $timeout = 300
+          $elapsed = 0
+          while ($elapsed -lt $timeout) {
+            try {
+              docker version 2>&1 | Out-Null
+              if ($LASTEXITCODE -eq 0) {
+                Write-Host "Docker is ready"
+                docker version
+                docker info
+                break
+              }
+            } catch {
+              Write-Host "Docker not ready yet... ($elapsed/$timeout seconds)"
+            }
+            Start-Sleep -Seconds 10
+            $elapsed += 10
+          }
+          if ($elapsed -ge $timeout) {
+            Write-Host "Docker failed to start within timeout"
+            exit 1
+          }
+
+      - name: Set up Docker (macOS)
+        if: runner.os == 'macOS'
+        run: |
+          brew install colima docker docker-compose
+          colima start --cpu 2 --memory 4
+          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 +89,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_DOCKER_SOCKET_OVERRIDE=/var/run/docker.sock" >> $GITHUB_ENV
+          echo "TESTCONTAINERS_HOST_OVERRIDE=localhost" >> $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 +123,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);