RFR: 8350546: Several java/net/InetAddress tests fails UnknownHostException [v10]

[Daniel Fuchs]

On Tue, 4 Mar 2025 14:34:35 GMT, SendaoYan <syan at openjdk.org> wrote:

>> Hi all,
>> 
>> Two java/net/InetAddress tests fails "java.net.UnknownHostException" on some special machines. The machine cannot connect to the Internet directly and needs to be connected to the Internet through a jump machine. The java/net/InetAddress tests report "java.net.UnknownHostException: bugs.openjdk.org: Temporary failure in name resolution" failure do not caused by JVM bug but environmentalk issue, so this PR match the java.net.UnknownHostException and throw jtreg.SkippedException instead of report test fails.
>> 
>> And run the tests with JVM options `-Dhttp.proxyHost=192.168.50.1 -Dhttp.proxyPort=10991 -Dhttps.proxyHost=192.168.50.1 -Dhttps.proxyPort=10991 -DsocksProxyHost=192.168.50.1 -DsocksProxyPort=10991` can not work around. So I create this PR to fix the test bug.
>> 
>> Command wget shows that the machine should connect to internet through a jumper machine(192.168.50.1:10991):
>> 
>>> wget www.bing.com
>> --2025-02-24 17:56:25--  http://www.bing.com/
>> Connecting to 192.168.50.1:10991... connected.
>> Proxy request sent, awaiting response... 301 Moved Permanently
>> Location: http://cn.bing.com/ [following]
>> --2025-02-24 17:56:25--  http://cn.bing.com/
>> Reusing existing connection to 192.168.50.1:10991.
>> Proxy request sent, awaiting response... 200 OK
>> Length: 13006 (13K) [text/html]
>> Saving to: ‘index.html’
>> 
>> 
>> Change has been verified locally, test-fix only and make tests more robustness, no risk.
>
> SendaoYan has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Update the comment and remove "import jtreg.SkippedException"

test/jdk/java/net/InetAddress/getOriginalHostName.java line 45:

> 43:         InetAddress ia = null;
> 44:         ia = getInetAddress(HOST);
> 45:         if (ia != null) testInetAddress(ia, HOST);

I noticed that's there is one missing test case. This should not throw UnknownHostException either.

Suggestion:

        ia = getInetAddress(HOST);
        if (ia != null) testInetAddress(ia, HOST);
        ia = InetAddress.getByAddress(HOST, new byte[] { 1, 2, 3, 4});
        testInetAddress(ia, HOST);

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/23767#discussion_r1980015932