On Wed, 11 Jun 2025 07:15:43 GMT, SendaoYan <s...@openjdk.org> wrote:
> Hi all, > > I think use `@requires (os.family == "windows")` is more elegant than thow > jtreg.SkippedException in test java/io/File/MaxPath.java. > > On linux before this PR, run this test jtreg report: > > > TEST: java/io/File/MaxPath.java > TEST RESULT: Passed. Skipped: jtreg.SkippedException: This test is run only > on Windows > > > On linux after this PR, run this test jtreg report: > > > Test results: no tests selected > > > Change has been verified locally both on linux and windows, test-fix only, no > risk. This pull request has now been integrated. Changeset: 5886ef72 Author: SendaoYan <s...@openjdk.org> URL: https://git.openjdk.org/jdk/commit/5886ef728fc1efe43e90e056c03725c3ee982ad6 Stats: 8 lines in 1 file changed: 0 ins; 7 del; 1 mod 8359182: Use @requires instead of SkippedException for MaxPath.java Reviewed-by: bpb, bchristi ------------- PR: https://git.openjdk.org/jdk/pull/25742