On Wed, 11 Oct 2023 15:45:29 GMT, Brian Burkhalter <b...@openjdk.org> wrote:
>> Change test to use `RandomFactory` instead of `new Random()` and convert it
>> to JUnit 5.
>
> Brian Burkhalter has updated the pull request incrementally with one
> additional commit since the last revision:
>
> 8313621: assertTrue -> assertEquals
test/jdk/jdk/internal/math/FloatingDecimal/TestFloatingDecimal.java line 65:
> 63: /**
> 64: * @test
> 65: * @bug 7032154
Suggestion:
* @bug 7032154 8313621
test/jdk/jdk/internal/math/FloatingDecimal/TestFloatingDecimal.java line 78:
> 76: */
> 77: public class TestFloatingDecimal {
> 78: private static final int NUM_RANDOM_TESTS = 100000;
Suggestion:
private static final int NUM_RANDOM_TESTS = 100_000;
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/16130#discussion_r1376266315
PR Review Comment: https://git.openjdk.org/jdk/pull/16130#discussion_r1376266653
