On Tue, 23 Jan 2024 13:15:46 GMT, Daniel Jeliński <[email protected]> wrote:
> 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. This pull request has now been integrated. Changeset: 95310eab Author: Daniel Jeliński <[email protected]> URL: https://git.openjdk.org/jdk/commit/95310eab6ce73512b1afc0a7a26a396dd7b6cb7c Stats: 25 lines in 1 file changed: 6 ins; 15 del; 4 mod 8223696: java/net/httpclient/MaxStreams.java failed with didn't finish within the time-out Reviewed-by: dfuchs ------------- PR: https://git.openjdk.org/jdk/pull/17536
