-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
Split the current "testing" chapter #111
Comments
I would gladly take care of this. Additionally, there is some misinformation to fix. |
\o/ Thanks!
Yeah, that could be better worded. What I mean is that |
There could be sections about:
Do you believe this would be useful or will it just bloat the section? |
I suppose this is to temporarily disable some unit tests? As an alternative to commenting out the test, the ignore attribute would be cleaner.
I didn't know this was possible! Is this baked in the test executable as a flag? Can I use this with benchmarks too? Sound like useful topics to me! I'll add them to the TODO list. |
The ignore attribute also permits ignoring tests by configuration, so platform-specific tests are possible, for example. Regex matching works for benchmarks and tests. You just pass the regular expression as a free argument to the test runner, and it will filter the tests by match. I will write those sections up once #159 is ok. |
I don't think @alxgnon is still working on this after a few years 😄 If someone else would like to pick up the torch, I'd be happy to mentor! |
I'd like to take on that. I think that it would be nice to have a chapter on testing (say, chapter 20 and moving meta and the rest after it to +1).
Looking for suggestions about improving this plan |
This sounds great! For the |
I'm closed this issue because #953 was merged |
The current chapter needs to be split:
#[test]
attribute andrustc --test
#[should_fail]
#[test]
is kind of sugar for#[cfg(test)]
ignore
attribute@alxgnon is working on this
The text was updated successfully, but these errors were encountered: