On Tue, 10 Oct 2023 23:21:42 GMT, Brian Burkhalter <b...@openjdk.org> wrote:
> Change test to use `RandomFactory` instead of `new Random()` and convert it
> to JUnit 5.
test/jdk/jdk/internal/math/FloatingDecimal/TestFloatingDecimal.java line 115:
> 113: }
> 114:
> 115: assertTrue(failures == 0);
Suggestion:
assertEquals(0, failures);
Gives a better message because it explicitly knows both the expected value (1st
arg) and the actual value (2nd arg).
Similarly for other `assertTrue` usages below).
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/16130#discussion_r1354633701
