[RFR] 8224645: Test java/nio/channels/DatagramChannel/BasicMulticastTests.java fails with NoSuchElementException

[Alan Bateman]

On 23/05/2019 18:19, Arthur Eubanks wrote:
> bug: https://bugs.openjdk.java.net/browse/JDK-8224645
> webrev: http://cr.openjdk.java.net/~aeubanks/8224645/webrev.00/
>
> Test java/nio/channels/DatagramChannel/BasicMulticastTests.java fails 
> with NoSuchElementException on Solaris.
>
> java.util.NoSuchElementException
> at java.base/java.util.Spliterators$1Adapter.next(Spliterators.java:688)
> at BasicMulticastTests.main(BasicMulticastTests.java:221)
> at 
> java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native 
> Method)
> at 
> java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62) 
>
> at 
> java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43) 
>
> at java.base/java.lang.reflect.Method.invoke(Method.java:567)
> at 
> com.sun.javatest.regtest.agent.MainActionHelper$AgentVMRunnable.run(MainActionHelper.java:298) 
>
> at java.base/java.lang.Thread.run(Thread.java:830)
>
> This changes the test to check if any multicast interfaces are 
> available for IPv4/IPv6 before testing one. This does have the effect 
> of not failing if no multicast interfaces available, which may or may 
> not be desirable.
Adding nio-dev as this is a DatagramChannel test.

Might be a bit more readable if you rename "multicasts" to 
"multicastInterfaces" but otherwise looks okay to me.

-Alan.