facebookincubator / below

A time traveling resource monitor for modern Linux systems
Apache License 2.0
1.04k stars 60 forks source link

Run tests inside each project independently. #8175

Closed brianc118 closed 1 year ago

brianc118 commented 1 year ago

We've had issues where CI did not catch a test breakage, e.g. from 0b24b4cd235ab8032a850498fd87d9673b01884f

cargo test --all isn't exactly the same as testing the individual nested projects. For example, if we remove all default features from the regex dep in below-common it will fail when you test with working on the project independently. However if you run cargo test on the top level project, it will pass because other nested projects also use regex but do include the default features.