Skip to content

ci: use nextest for test suites - #150

Merged
brndnmtthws merged 1 commit into
mainfrom
brndnmtthws/ci/use-nextest
Jul 22, 2026
Merged

ci: use nextest for test suites#150
brndnmtthws merged 1 commit into
mainfrom
brndnmtthws/ci/use-nextest

Conversation

@brndnmtthws

Copy link
Copy Markdown
Owner

Summary

Use cargo-nextest for executable test suites in the build and publish workflows.

User Impact

CI and release validation now use the same faster test runner consistently across native, feature-matrix, MSRV, nightly, and WebAssembly jobs. Doctest coverage remains unchanged.

Root Cause

Several workflow jobs still invoked cargo test directly and did not install cargo-nextest, leaving test execution inconsistent across CI.

Fix

Install cargo-nextest with authenticated GitHub requests in each affected job and replace executable cargo test runs with cargo nextest run. Keep cargo test --doc because nextest does not support doctests.

Validation

  • mise x actionlint@1.7.12 shellcheck@0.11.0 -- actionlint .github/workflows/build-and-test.yml .github/workflows/publish.yml
  • cargo nextest run --target wasm32-unknown-unknown --no-default-features --features serde,base64,wincode (5 passed)
  • cargo nextest run --all-targets (298 passed)
  • cargo +nightly nextest run --all-targets --all-features (334 passed)
  • git diff --check HEAD^

@netlify

netlify Bot commented Jul 22, 2026

Copy link
Copy Markdown

Deploy Preview for dryoc-docs ready!

Name Link
🔨 Latest commit c519c3d
🔍 Latest deploy log https://app.netlify.com/projects/dryoc-docs/deploys/6a60b43f1718470008f8975e
😎 Deploy Preview https://deploy-preview-150--dryoc-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@brndnmtthws
brndnmtthws marked this pull request as ready for review July 22, 2026 12:15
Copilot AI review requested due to automatic review settings July 22, 2026 12:15

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@brndnmtthws
brndnmtthws merged commit 8a3ac0d into main Jul 22, 2026
33 checks passed
@brndnmtthws
brndnmtthws deleted the brndnmtthws/ci/use-nextest branch July 22, 2026 12:21
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