RFR: 7903961: Make default timeout configurable
Christian Stein
cstein at openjdk.org
Mon Mar 10 17:43:10 UTC 2025
On Fri, 7 Mar 2025 10:45:30 GMT, Christian Stein <cstein at openjdk.org> wrote:
> Please review this change replacing the hard-coded `120` seconds default timeout for test actions with a configurable solution parsing a custom default value from `TEST.ROOT` and `TEST.properties` files:
>
>
> timeout.default.seconds=360
>
>
> Especially in combination with "JUnit.dirs" and "TestNG.dirs", where all actions directives found in a test file are ignored, is helpful.
src/share/classes/com/sun/javatest/regtest/exec/RegressionScript.java line 1131:
> 1129: Map<String, String> getTestProperties() {
> 1130: // initialize the properties with standard properties common to all tests
> 1131: Map<String, String> p = new TreeMap<>(params.getBasicTestProperties());
This change sorts all basic test properties by their name; keeping the insertion order was never a promise.
-------------
PR Review Comment: https://git.openjdk.org/jtreg/pull/253#discussion_r1987751737
More information about the jtreg-dev
mailing list