On Fri, 25 Apr 2025 10:18:37 GMT, Mark Sheppard <mshep...@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

Reply via email to