RFR: 8223696: java/net/httpclient/MaxStreams.java failed with didn't finish within the time-out
Daniel Jeliński
djelinski at openjdk.org
Tue Jan 23 13:48:33 UTC 2024
The test was occasionally failing with TestNG timeout. When a TestNG test times out, TestNG interrupts the test thread, and starts another test case in a new thread. The timeout is invisible to JTReg, and the timeout failure handlers are not run.
This PR removes the TestNG timeout. It also removes `-ea -esa` from the test command line (the test does not use `assert`), and reduces the amount of synchronization objects used.
I verified that the timeouts are now handled by JTReg. The test still passes.
-------------
Commit messages:
- Remove TestNG timeout
Changes: https://git.openjdk.org/jdk/pull/17536/files
Webrev: https://webrevs.openjdk.org/?repo=jdk&pr=17536&range=00
Issue: https://bugs.openjdk.org/browse/JDK-8223696
Stats: 24 lines in 1 file changed: 5 ins; 15 del; 4 mod
Patch: https://git.openjdk.org/jdk/pull/17536.diff
Fetch: git fetch https://git.openjdk.org/jdk.git pull/17536/head:pull/17536
PR: https://git.openjdk.org/jdk/pull/17536
More information about the net-dev
mailing list