Skip to content

Comments

ci: Add VM runtime fixes for E2E smoke tests#68

Merged
dtkav merged 1 commit intomainfrom
fix/e2e-runtime-fixes
Feb 14, 2026
Merged

ci: Add VM runtime fixes for E2E smoke tests#68
dtkav merged 1 commit intomainfrom
fix/e2e-runtime-fixes

Conversation

@mgmobrien
Copy link
Member

Summary

  • Start Xvfb virtual display before test execution (Obsidian needs a display, even headless)
  • Clean /tmp/test-reports before each run (prevents stale results from previous runs)
  • Install playwright deps before running tests (npm install --ignore-scripts)
  • Fix verdict logic: zero tests executed now correctly reports FAIL instead of PASS

Context

These fixes were on cf/e2e-smoke-ssh-fix but that branch was merged before they were included. Re-applied to current main.

Validation

Full test-branch.sh origin/main ran on relay-e2e-linux as runner user: 7 pass, 1 unexpected fail (product bug), 1 expected fail (known bug). The pipeline works end-to-end.

Test plan

  • Trigger workflow_dispatch on this branch and verify tests actually execute
  • Verify zero-test scenario reports FAIL (not PASS)

🤖 Generated with Claude Code

- Start Xvfb virtual display before test execution (headless Obsidian)
- Clean stale test reports before each run
- Install playwright deps before running tests
- Fix verdict logic: zero tests executed now correctly reports FAIL

These fixes were validated by running the full test-branch.sh pipeline
on relay-e2e-linux as the runner user (7 pass, 1 unexpected fail,
1 expected fail).

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@dtkav dtkav merged commit 179d35e into main Feb 14, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants