RFR: 8281511: java/net/ipv6tests/UdpTest.java fails with checkTime failed [v7]

Daniel Fuchs dfuchs at openjdk.org
Tue Mar 4 18:28:53 UTC 2025


On Tue, 4 Mar 2025 18:23:42 GMT, Daniel Fuchs <dfuchs at openjdk.org> wrote:

>> serhiysachkov has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   8281511: review comments updates
>
> test/jdk/java/net/ipv6tests/Tests.java line 161:
> 
>> 159:         if (got < expected) {
>> 160:             throw new RuntimeException("checkIfTimeOut failed: got " + got
>> 161:                     + ", expected at least " + expected );
> 
> `checkTime` is also called in `TcpTests`. You will need to modify that test and verify that the new semantics applies there too.

I see that `TcpTests` also uses `System.currentTimeMillis` so you could take the opportunity to fix that too.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/23840#discussion_r1979991904


More information about the net-dev mailing list