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