On Thu, 11 Sep 2025 09:27:06 GMT, Alan Bateman <al...@openjdk.org> wrote:
>> This PR >> * Moves sqrt() and nthRoot() tests to a new subset. >> * Removes a leftover `@run` tag without a timeout. >> * Increases the explicit timeout from 400 to 480. >> * Sorts the jtreg tags according to standard conventions. > > test/jdk/java/math/BigInteger/BigIntegerTest.java line 32: > >> 30: * @library /test/lib >> 31: * @build jdk.test.lib.RandomFactory >> 32: * @run main/timeout=480 BigIntegerTest > > Should the timeoutFactor change have adjusted this from 400 to 1600? There were two `@run` lines, one without a timeout (which is now removed), and a later one with a timeout of 400. IIUC, this means that the test was executed 2 times, one with a default timeout, and another with the explicit timeout of 400. The JBS issue shows a timeout after 120 seconds, so I guess this was due to the first `@run` line without an explicit timeout. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/27211#discussion_r2339770690
