-
Notifications
You must be signed in to change notification settings - Fork 1.1k
Initial gcov work #219
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
Initial gcov work #219
Conversation
…d fuzz tests explicit and prevent accidental randomization of deterministic tests
…erator state for bignum & openssl tests
|
Just a quick question I had: Whats the motivation of not running the randomized tests in coverage testing? They're just as deterministic as the rest when run with a static seed. |
|
I think that could get challenging to maintain in practice. In order to guarantee identical test behavior across changes, it'd be necessary to not only never change the RNG algorithm, but also not change the sequence of calls to the RNG (as could happen with adding, removing, or changing the order of tests, for example). Or, statically seed before every test and make sure the intra-test RNG call pattern doesn't change. Also, for me personally, I find a fixed sequence of instructions used to evoke a specific condition in the code under test is easier to reason confidently about. There's philosophical motivations as well, upon which I'm happy to expound if you like. :-) |
|
P.S. I'm happy to take a different approach if you prefer. |
|
No worries, let me know when you're ready. |
Separation of deterministic tests from randomized tests; initial work for gcov build & run. Next step is filling out deterministic tests to trigger branches. All feedback welcome.