RFR: 8365983: Tests should throw SkippedException when SCTP not supported [v4]
Jaikiran Pai
jpai at openjdk.org
Tue Sep 2 07:08:46 UTC 2025
On Tue, 2 Sep 2025 07:04:03 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.
>
> SendaoYan has updated the pull request incrementally with one additional commit since the last revision:
>
> Remove extra blank line
These test-only changes look OK to me.
-------------
Marked as reviewed by jpai (Reviewer).
PR Review: https://git.openjdk.org/jdk/pull/26903#pullrequestreview-3175155396
More information about the net-dev
mailing list