[nextest-runner] add cargo nextest store export#3010
Conversation
There was a problem hiding this comment.
Pull request overview
Adds support for creating a portable “recorded run” archive (zip) via cargo nextest store archive, including runner-side archive creation, CLI plumbing, and CI artifact upload.
Changes:
- Implement portable archive writer in
nextest-runner(manifest +run.log.zst+store.zip). - Add
cargo nextest store archivesubcommand, wiring errors + exit codes. - Extend integration tests and CI workflow to exercise and upload the archive.
Reviewed changes
Copilot reviewed 10 out of 10 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
| nextest-runner/src/record/store.rs | Adds StoreRunsDir::new constructor to allow external creation of StoreRunsDir. |
| nextest-runner/src/record/portable.rs | New portable archive writer + unit tests for archive contents and error cases. |
| nextest-runner/src/record/mod.rs | Registers portable module and re-exports archive writer/result types. |
| nextest-runner/src/record/format.rs | Adds portable archive manifest format types and version constant. |
| nextest-runner/src/errors.rs | Introduces PortableArchiveError for archive creation failures. |
| cargo-nextest/src/dispatch/utility/store.rs | Adds store archive command implementation and CLI args. |
| cargo-nextest/src/errors.rs | Plumbs PortableArchiveError through ExpectedError with a dedicated exit code. |
| cargo-nextest/src/dispatch/core/tests.rs | Adds argument parsing coverage for the new store archive command. |
| integration-tests/tests/integration/record_replay.rs | Verifies portable archive creation and validates archive contents/manifest structure. |
| .github/workflows/ci.yml | Enables recording in CI, creates a portable archive, uploads it as a build artifact. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Codecov Report❌ Patch coverage is Additional details and impacted files@@ Coverage Diff @@
## main #3010 +/- ##
==========================================
+ Coverage 83.10% 83.21% +0.11%
==========================================
Files 154 155 +1
Lines 40919 41301 +382
==========================================
+ Hits 34004 34370 +366
- Misses 6915 6931 +16 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
sunshowers
force-pushed
the
archive
branch
2 times, most recently
from
f3da11e to
745e7cd
Compare
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 13 out of 13 changed files in this pull request and generated 1 comment.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
sunshowers
force-pushed
the
archive
branch
5 times, most recently
from
da25728 to
9d2c0f3
Compare
sunshowers
changed the title
sunshowers
force-pushed
the
archive
branch
2 times, most recently
from
4c79ff0 to
97ef6e2
Compare
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 13 out of 13 changed files in this pull request and generated 1 comment.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 14 out of 14 changed files in this pull request and generated no new comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
We can just through events by accepting a fallible iterator.
No description provided.