Skip to content

Commit 005feb6

Browse files
bgrgicakclaude
andcommitted
Run per-engine E2E tests in CI with explicit --engine flags
Switch CI E2E jobs from sandbox.e2e.ts (auto-detected engine) to sandbox-engines.e2e.ts (explicit --engine docker/podman/containerd). Each job now verifies the specific engine it's responsible for rather than relying on auto-detection matching the intended runtime. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
1 parent 98f2769 commit 005feb6

1 file changed

Lines changed: 3 additions & 5 deletions

File tree

.github/workflows/ci.yml

Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -39,7 +39,7 @@ jobs:
3939
- run: npm ci
4040
- run: npx tsc --build
4141
- name: E2E tests (sandbox with Docker)
42-
run: npx tsx --test 'packages/cli/test/e2e/sandbox.e2e.ts'
42+
run: npx tsx --test 'packages/cli/test/e2e/sandbox-engines.e2e.ts'
4343

4444
e2e-podman:
4545
runs-on: ubuntu-latest
@@ -53,12 +53,11 @@ jobs:
5353
- run: npm ci
5454
- run: npx tsc --build
5555
- name: E2E tests (sandbox with Podman)
56-
run: npx tsx --test 'packages/cli/test/e2e/sandbox.e2e.ts'
56+
run: npx tsx --test 'packages/cli/test/e2e/sandbox-engines.e2e.ts'
5757

5858
e2e-colima:
5959
runs-on: macos-latest
6060
# Colima needs nested virtualization which macOS CI runners don't reliably support.
61-
# This job validates the install+start flow but may fail on VM creation.
6261
continue-on-error: true
6362
steps:
6463
- uses: actions/checkout@v4
@@ -72,10 +71,9 @@ jobs:
7271
- run: npm ci
7372
- run: npx tsc --build
7473
- name: E2E tests (sandbox with Colima)
75-
if: success()
7674
run: |
7775
if colima status 2>/dev/null; then
78-
npx tsx --test 'packages/cli/test/e2e/sandbox.e2e.ts'
76+
npx tsx --test 'packages/cli/test/e2e/sandbox-engines.e2e.ts'
7977
else
8078
echo "Skipping: Colima is not running"
8179
fi

0 commit comments

Comments
 (0)