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