RFR: 8355444: [java.io] Use @requires tag instead of exiting based on "os.name" property value

Brian Burkhalter bpb at openjdk.org
Fri Apr 25 16:07:45 UTC 2025


On Fri, 25 Apr 2025 10:18:37 GMT, Mark Sheppard <msheppar at openjdk.org> wrote:

> When running test locally that's fine, this is easily seen in the jtreg report, but when run in the test frameworks CI pipeline, this "not run " visibility is obscured. Thus when doing test run audit of a jobs, requires a bit more effort to locate test that have not been run.

Maybe the visibility in CI testing should be addressed?

> An alternative practice has been to throw a SkippedException and this is more easily observed in a pipeline job's results, as the SkippedException are part of all results, and jtreg now reports Skipped tests in its summary

Do you think it would be better overall to use `SkippedException` instead of `@requires`, or only in certain cases, e.g., `MacPathTest`, etc.?

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

PR Comment: https://git.openjdk.org/jdk/pull/24860#issuecomment-2830826685


More information about the core-libs-dev mailing list