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