Skip to content

feat: enable listing with cargo-nextest and a timeout #290

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

Ekleog-Polygon
Copy link
Contributor

Builds on top of #289, review only the second commit to see the changes introduced here.

This allows using nextest with a specific nextest profile, instead of cargo test. In particular, with this nextest configuration:

[profile.list-fuzzers]
slow-timeout = { period = "500ms", terminate-after = 1 }

I'm getting the following improvements on listing fuzzers for https://github.com/agglayer/agglayer.
Before:

cargo  57.07s user 10.98s system 92% cpu 1:13.75 total

After:

cargo  8.38s user 3.77s system 47% cpu 25.509 total

Which is a ~3x speed improvement, without any fuzzer being missed (because fuzzers naturally reply to CARGO_BOLERO_SELECT in less than 500ms) 🎉

I'm thinking this could be very helpful for large repos, at least until something like #196 could get implemented.

WDYT?

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.

1 participant