RFR: 8367365: java/math/BigInteger/BigIntegerTest.java failed in jtreg timeout

[Raffaello Giulietti]

On Thu, 11 Sep 2025 09:27:06 GMT, Alan Bateman <alanb at 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