Why not make the array 1000 items long? Gary
On Tue, Feb 27, 2018 at 10:31 AM, Allon Mureinik <murei...@gmail.com> wrote: > All the ArrayUtilsTest#testShuffleXYZ tests take an array, shuffle it, and > assert that the result isn't equal to the original array. > This is usually true, but there's a small chance that the shuffled array > will be equal to the original array, and thus the test will fail. This > chance is higher for the testShuffleBoolean case where the array contains > ten elements, but only two distinct values (true and false). > > I've sent a PR to remove these problematic assertions, let's see what the > maintainers think of it: > https://github.com/apache/commons-lang/pull/316 > > > On Tue, Feb 27, 2018 at 6:59 PM, Otto Fowler <ottobackwa...@gmail.com> > wrote: > > > Note, this does pass in my personal travis: > > https://travis-ci.org/ottobackwards/commons-lang/builds/346806991 > > > > > > On February 27, 2018 at 11:58:24, Otto Fowler (ottobackwa...@gmail.com) > > wrote: > > > > My PR is currently failing for java 9 on this test. Anyone have any idea > > why? > > > > [INFO] Running org.apache.commons.lang3.ArrayUtilsTest > > [ERROR] Tests run: 307, Failures: 1, Errors: 0, Skipped: 0, Time elapsed: > > 0.114 s <<< FAILURE! - in org.apache.commons.lang3.ArrayUtilsTest > > [ERROR] testShuffleBoolean(org.apache.commons.lang3.ArrayUtilsTest) Time > > elapsed: 0.008 s <<< FAILURE! > > java.lang.AssertionError > > at > > org.apache.commons.lang3.ArrayUtilsTest.testShuffleBoolean( > > ArrayUtilsTest.java:5023) > > >
