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

Mark Sheppard msheppar at openjdk.org
Fri Apr 25 10:21:01 UTC 2025


On Thu, 24 Apr 2025 20:46:42 GMT, Brian Burkhalter <bpb at openjdk.org> wrote:

> Use the `@requires` tag instead of obtaining the operating system name from the `os.name` property and then exiting if the test is not run on that operating system.

a general comment on this approach of running tests selectively on an OS via @requires
It is all cool and dandy, but this has implications on the visibility of the test not being run. 
It will be added to the jtreg "not run" list
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.

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

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

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


More information about the core-libs-dev mailing list