Make the rand tests determinstic. #2796

pull gmaxwell wants to merge 1 commits into bitcoin:master from gmaxwell:remove_overactive_rand_tests changing 1 files +1 −33

gmaxwell commented at 7:10 PM on June 25, 2013: contributor

This avoids spurious errors with the old tests but still tests enough that if the rng is replaced with a totally broken one it should still fail.
TheBlueMatt commented at 12:57 PM on July 7, 2013: member

Huh? The removes the tests that are deterministic and leaves only the ones that cause spurious errors on pull-tester every once in a while.
gmaxwell commented at 3:03 PM on July 7, 2013: contributor

@TheBlueMatt No it doesn't. I promise.
TheBlueMatt commented at 3:09 PM on July 7, 2013: member

Ahhh, yes...can you add (fixes #2714) to the commit message so it gets closed on merge?
TheBlueMatt commented at 4:53 PM on July 7, 2013: member

ACK

Make the rand tests determinstic. (fixes #2714)

This avoids spurious errors with the old tests but still tests
 enough that if the rng is replaced with a totally broken one
 it should still fail.

e5c4dfdfc0

Contributors