peterolson / BigInteger.js / 147

94%

Pull #112

travis-ci

Fix randBetween bias and range issues

This corrects two problems:

1. When the min-max range was "small", the min and max integers had
   only half the probability of any of the numbers in between. This
   was due to how the random value was rounded, which effectively
   increased the integer output by 1 half of the time.

   This was fixed by flooring the value instead, and increasing the
   range by 1 to cover the max integer.

2. When the range was "big", the max value was treated as exclusive,
   which contradicts how it behaves in "small" mode and how
   randBetween is used internally. It is now treated as inclusive.

426 of 445 branches covered (95.73%)

2 of 2 new or added lines in 1 file covered. (100.0%)

828 of 832 relevant lines covered (99.52%)

201166.56 hits per line