> Can I get a review of this test-only fix which addresses the intermittent > failrues in `com/sun/net/httpserver/HttpsParametersClientAuthTest.java` on > Windows? > > As noted in https://bugs.openjdk.org/browse/JDK-8331334, the test relies on > the TLS communication (of a HTTP request) to fail and then asserts on the > exception types. On Windows, sometimes the exception type differs, although > the underlying failure is the expected failure. > > The commit in this PR adds an additional condition to allow for `IOException` > to be thrown when on Windows. The exception type and message check is > additionally only added for Windows and the message check is very specific > intentionally, to make sure we don't allow any `IOException` to result in the > test passing. The usage of the `WSAECONNABORTED_MSG` has a precedent in > `test/jdk/java/net/httpclient/HandshakeFailureTest.java`, so I think that > should be OK in this test too. > > With these changes the test hasn't failed in a test repeat of 100. Without > the changes, the test fails around 15 times in a test repeat of 100. > Additionally tier1, tier2 and tier3 tests have been run with this change and > those tests continue to pass.
Jaikiran Pai has updated the pull request incrementally with one additional commit since the last revision: don't check for exception message (which can be localized) ------------- Changes: - all: https://git.openjdk.org/jdk/pull/19091/files - new: https://git.openjdk.org/jdk/pull/19091/files/8cd9f836..403e2acb Webrevs: - full: https://webrevs.openjdk.org/?repo=jdk&pr=19091&range=01 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=19091&range=00-01 Stats: 11 lines in 1 file changed: 0 ins; 4 del; 7 mod Patch: https://git.openjdk.org/jdk/pull/19091.diff Fetch: git fetch https://git.openjdk.org/jdk.git pull/19091/head:pull/19091 PR: https://git.openjdk.org/jdk/pull/19091
