-
Notifications
You must be signed in to change notification settings - Fork 2.5k
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
Filter and build integration tests needed only #8282
Comments
Thanks for the report, but |
Ah, I did not know |
I think this is already documented in the |
Ah, I just realized it is documented in |
Mind if I take a stab at this? I haven't been able to contribute to an OSS project before so I'm trying to find ways to help out. It looks like I should be able to add specific wording in |
@icorbrey Yes, definitely. It would be good to make the wordings simpler and clearer. One thing I like is https://github.com/Dhghomon/easy_rust |
I think that adding the rust manuals to the |
Describe the problem you are trying to solve
There is a project with 20+ integration tests, it takes like few minutes to build all of them. Even with a small change in one of the tests file and I wanted to just run one of the tests file, I did
cargo tests part_a
but it still compiles everything again.The solution right now,
rm tests/*
git restore tests/part_a.rs
cargo test part_a
, try rebuildinggit restore src/dep.rs
, add some dependencies neededEssentially, this will also remove the issue of clogging the screen with 0 tests run and the need to scroll up multiple times to search for the tests run #4324.
Describe the solution you'd like
Search for the functions used in the filter (
cargo test <filter>
), build and run only those files needed. Something like ripgrep for function name, but this requires cargo to include the dependencies too.This provides 2 benefits, reducing the need to build unneeded files and prevent clogging the screen by running file without tests matching the filter.
Notes
I would like to take on this issue if the complexity is not high or if there are people willing to mentor.
The text was updated successfully, but these errors were encountered: