Skip to content
This repository has been archived by the owner on Dec 12, 2024. It is now read-only.

👷 Use cargo nextest to run tests in CI #25

Merged
merged 3 commits into from
Dec 9, 2024

Conversation

christeefy
Copy link
Collaborator

@christeefy christeefy commented Dec 8, 2024

Summary & Motivation

cargo nextest is a runner that's produces more readable test results compared to cargo test. This makes it easier to identify which test failed. It's supposedly faster too.

Below is a comparison of both outputs.

cargo test

image

cargo nextest run

image

How I Tested These Changes

  • CI passes

Changelog

Ensure that an entry has been created in CHANGELOG.md outlining additions, deletions, and/or modifications.

See: keepachangelog.com

@christeefy
Copy link
Collaborator Author

christeefy commented Dec 8, 2024

Ah just realized that nextest doesn't support running doctests (nextest-rs/nextest#16).
So doctests would need to be run as a separate step (with no additional compilation overhead).

@christeefy christeefy marked this pull request as ready for review December 8, 2024 12:52
Copy link
Owner

@cmpadden cmpadden left a comment

Choose a reason for hiding this comment

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

Looks good!
image

@cmpadden cmpadden merged commit a289cf9 into cmpadden:main Dec 9, 2024
2 checks passed
@christeefy christeefy deleted the ci/use-nextest branch December 9, 2024 14:57
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants