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.

Marked as reviewed by bchristi (Reviewer).

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

PR Review: https://git.openjdk.org/jdk/pull/25742#pullrequestreview-2918913804

Reply via email to