Integrated: 8223696: java/net/httpclient/MaxStreams.java failed with didn't finish within the time-out

[Daniel Jeliński]

On Tue, 23 Jan 2024 13:15:46 GMT, Daniel Jeliński <djelinski at openjdk.org> 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 <djelinski at openjdk.org>
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