RFR: 7903961: Make default timeout configurable
Christian Stein
cstein at openjdk.org
Thu Apr 3 10:46:20 UTC 2025
On Tue, 1 Apr 2025 09:29:47 GMT, Jaikiran Pai <jpai at openjdk.org> wrote:
>> 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.
>
> Does the new proposed feature of configuring a default timeout continue to work without this change to using a `TreeMap`? If yes, then I think it would be good to separate this change out from this PR.
Good point. Reverting this change.
-------------
PR Review Comment: https://git.openjdk.org/jtreg/pull/253#discussion_r2026723571
More information about the jtreg-dev
mailing list