Skip to content

Run tests with cargo-nextest#367

Merged
jdm merged 1 commit into
servo:mainfrom
mrobinson:run-tests-with-nexttest
Jun 7, 2026
Merged

Run tests with cargo-nextest#367
jdm merged 1 commit into
servo:mainfrom
mrobinson:run-tests-with-nexttest

Conversation

@mrobinson

@mrobinson mrobinson commented Jun 7, 2026

Copy link
Copy Markdown
Member

This is a speculative fix to try to make test runs more reliable. The
inspiration here is wgpu which also uses cargo-nextest to run tests.
This will isolate each test by process, which should help to avoid
interference between them.

As part of this change, we also need to fix an issue in one of the tests
where a worker thread can outlive the test that spawned it. The fix
is to wait until the worker thread joins the main thread before finishing
the test, otherwise it will cause an error in test execution.

@mrobinson mrobinson force-pushed the run-tests-with-nexttest branch 3 times, most recently from dc78494 to 5bd75be Compare June 7, 2026 19:57
This is a speculative fix to try to make test runs more reliable. The
inspiration here is wgpu which also uses `cargo-nextest` to run tests.
This will isolate each test by process, which should help to avoid
interference between them.

Signed-off-by: Martin Robinson <martin@abandonedwig.info>
@jdm jdm added this pull request to the merge queue Jun 7, 2026
Merged via the queue into servo:main with commit 110a5c5 Jun 7, 2026
29 checks passed
@mrobinson mrobinson deleted the run-tests-with-nexttest branch June 7, 2026 20:37
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