test: support isolated parallel desktop E2E workers - #498
Merged
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Related Issue
Closes #495
Verification
pnpm test:e2e:desktop:runnervalidates option parsing, worker bounds, ordered scenario groups, and healthy-worker continuation after failure.pnpm build:e2e:desktopbuilds the E2E Tauri application with its test-only runtime identifier override.pnpm checkpasses the cross-cutting frontend, backend, formatting, lint, build, coverage, and Rust test gates.pnpm check:frontendrevalidates the final TypeScript runner and documentation state after cleanup hardening.cargo clippy --all-targets --features desktop-e2e --manifest-path src-tauri/Cargo.toml -- -D warningsvalidates the desktop E2E Rust feature path.Manually verified in Windows:
--workers 1and--workers 2; a representative run took about 123 seconds with one worker and 74 seconds with two.Not verified: concurrent execution on a GitHub-hosted Windows runner. CI intentionally requests
--workers 1until the pull request run provides comparable hosted reliability and timing evidence.Notes
The default remains one worker. Local callers can opt into two through four workers without changing scenario coverage; switching CI later is a one-line worker-count change once hosted evidence supports it.