ci: run e2e tests on macOS and Windows; gate Docker-based tests to Linux

- e2e job uses OS matrix: ubuntu-latest, macos-latest, windows-latest
- Use cargo test --test "*" for cross-platform instead of make
- Windows setup: enable git long paths; set CMAKE_POLICY_VERSION_MINIMUM=3.5 for aws-lc-sys
- tests: gate Docker/testcontainers tests (airgap, push) to Linux and add no-op stubs elsewhere

Signed-off-by: Rishi Jat <[email protected]>

Author: rishi-jat

.github/workflows/ci.yml

@@ -63,15 +63,31 @@ jobs:
 
   e2e-tests:
     name: E2E tests
-    runs-on: ubuntu-latest
+    runs-on: ${{ matrix.os }}
+    strategy:
+      matrix:
+        os: [ubuntu-latest, macos-latest, windows-latest]
     steps:
+      - name: enable git long paths on Windows
+        if: matrix.os == 'windows-latest'
+        run: git config --global core.longpaths true
+
       - uses: actions/checkout@08c6903cd8c0fde910a37f88322edcfb5dd907a8 # v5.0.0
+
       - uses: sigstore/cosign-installer@d58896d6a1865668819e1d91763c7751a165e159 # v3.9.2
         # FIXME workaround for https://github.com/kubewarden/kwctl/issues/1245
         with:
           cosign-release: "v2.5.0"
+
+      # aws-lc-sys CMakefile contains a directive that has been removed from
+      # cmake v4 (released Mar 2025). The build failure can be fixed by setting an env var
+      - name: fix aws-lc-sys building with cmake 4.0.0 (Windows)
+        if: matrix.os == 'windows-latest'
+        shell: pwsh
+        run: echo 'CMAKE_POLICY_VERSION_MINIMUM=3.5' >> $env:GITHUB_ENV
+
       - name: run e2e tests
-        run: make e2e-tests
+        run: cargo test --test "*"
 
   coverage:
     name: coverage

tests/airgap.rs

@@ -1,10 +1,15 @@
 use assert_cmd::Command;
 use std::path::{Path, PathBuf};
 use tempfile::tempdir;
+
+// Only import and use testcontainers on Linux runners where Docker is available
+#[cfg(target_os = "linux")]
 use testcontainers::{core::WaitFor, runners::SyncRunner};
 
 mod common;
 
+// Run this test only on Linux where Docker/testcontainers is available
+#[cfg(target_os = "linux")]
 #[test]
 fn test_airgap() {
     let tempdir = tempdir().unwrap();
@@ -101,6 +106,13 @@ fn test_airgap() {
         .success();
 }
 
+// Provide a harmless stub on non-Linux platforms so the suite passes
+#[cfg(not(target_os = "linux"))]
+#[test]
+fn test_airgap() {
+    eprintln!("skipping test_airgap on non-Linux platforms");
+}
+
 fn setup_airgap_script_command(script: &Path, tempdir: &Path) -> Command {
     let mut cmd = Command::new(script);
 

tests/e2e.rs

@@ -17,6 +17,8 @@ use predicates::{prelude::*, str::contains, str::is_empty};
 use rstest::rstest;
 use sha2::{Digest, Sha256};
 use tempfile::tempdir;
+// Only import testcontainers on Linux (Docker availability)
+#[cfg(target_os = "linux")]
 use testcontainers::{core::WaitFor, runners::SyncRunner};
 
 mod common;
@@ -696,6 +698,8 @@ fn test_save_and_load() {
     }
 }
 
+// Run push test only on Linux where Docker/testcontainers is available
+#[cfg(target_os = "linux")]
 #[test]
 fn test_push() {
     let registry_image = testcontainers::GenericImage::new("docker.io/library/registry", "2")
@@ -780,6 +784,13 @@ fn test_push() {
         .stdout(contains("my-pod-privileged-policy:v0.1.10"));
 }
 
+// Provide a harmless stub on non-Linux platforms so the suite passes
+#[cfg(not(target_os = "linux"))]
+#[test]
+fn test_push() {
+    eprintln!("skipping test_push on non-Linux platforms");
+}
+
 #[rstest]
 #[case::pull_policies_before_scaffold(true)]
 #[case::pull_policies_on_demand(false)]