RFR: 8365983: Tests should throw SkippedException when SCTP not supported
ExE Boss
duke at openjdk.org
Fri Aug 22 23:56:52 UTC 2025
On Fri, 22 Aug 2025 13:30:19 GMT, SendaoYan <syan at openjdk.org> wrote:
> Currently, take test/jdk/com/sun/nio/sctp/SctpMultiChannel/Branch.java as example. When Util.isSCTPSupported() return false, then test print a warning and the exit.
>
> I think it's better to throw jtreg.SkippedException rather than only print a warning and then report test passed.
>
> Change has been verified locally, test-fix only, almost no risk.
test/jdk/com/sun/nio/sctp/SctpMultiChannel/Branch.java line 2:
> 1: /*
> 2: * Copyright (c) 2009, 2025 Oracle and/or its affiliates. All rights reserved.
Missing comma:
Suggestion:
* Copyright (c) 2009, 2025, Oracle and/or its affiliates. All rights reserved.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/26903#discussion_r2294949015
More information about the nio-dev
mailing list