On Mon, 30 Jun 2025 16:52:55 GMT, Jaikiran Pai <j...@openjdk.org> wrote:
>> Can I please get a review of this test-only change which addresses the issue >> noted in https://bugs.openjdk.org/browse/JDK-8359337? >> >> On macOS, the JDK by default (without any explicit system properties) picks >> up the proxy settings configured on the host. These proxy settings then get >> used by the JVM wide `DefaultProxySelector`. That can then result in some >> tests, like the ones listed in that issue, to fail. >> >> The commit in this PR updates these tests to disable the proxy when running >> these tests. I've verified that after these changes, these tests now pass on >> the host where these tests were failing previously. Additionally I've run >> tier2 to make sure nothing else is impacted due to this change. > > Jaikiran Pai has updated the pull request incrementally with one additional > commit since the last revision: > > Joe's review - add bugid Hello Johannes, > What is the error seen on macOS when the proxy is not disabled? These tests have been written to trigger specific error conditions. Some of them rely on a specific exception being thrown when communicating with an address that is known to be invalid. When a proxy is involved, the communication with the target address happens over the proxy which results in a different error (which depends on the proxy implementation). That causes the assertions in these tests to fail. Thank you all for the reviews. I'll go ahead and integrate this now. ------------- PR Comment: https://git.openjdk.org/jdk/pull/26052#issuecomment-3023572131 PR Comment: https://git.openjdk.org/jdk/pull/26052#issuecomment-3023573558