fix: fix `cargo bolero list` when some tests are failing

camshaft / bolero

property testing and verification front-end for Rust

https://camshaft.github.io/bolero

MIT License

185 stars 17 forks source link

fix: fix `cargo bolero list` when some tests are failing #195

Closed Ekleog-NEAR closed 11 months ago

Ekleog-NEAR commented 11 months ago

Before this change, cargo bolero list would ignore any fuzzers defined after a binary with one failing test. With this change, it properly lists all fuzzers.

This was prompted by my discovering our fuzzing infra’s builds with cargo bolero build-clusterfuzz had empty fuzzer tarballs, because one test was failing before the fuzzers were discovered.

I also added a regression test.

camshaft commented 11 months ago

Thanks!

Ekleog-NEAR commented 11 months ago

Thank you for the quick merge! :)

A proper fix would be https://github.com/camshaft/bolero/issues/196 (I couldn’t find the place where we had already discussed this), but hopefully this’ll make do in the meantime :)