alt Hacker NewsNot a good idea for CI tests. It will just make things flaky and gum up your PR/release process. Randomness or any form of nondeterminism should be in a different set of fuzzing tests (if you must use an RNG, a deterministic one is fine for CI).
Replies
dathinab • today at 4:23 PM
if it makes thing flaky
then it actually is a huge success
because it found a bug you overlooked in both impl. and tests
at least iff we speak about unit tests
➕ show 3 replies
That's why it's "randomInt(1,42)", not "randomLong()".