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

Reply via email to