On Wed, 23 Apr 2025 09:35:25 GMT, Manukumar V S <m...@openjdk.org> wrote:
> Fix the wrong usage of PassFailJFrame.forcePass() in some manual tests from > java.awt.Desktop package. These tests should be fixed similar to > [JDK-8352109](https://bugs.openjdk.org/browse/JDK-8352109) and > jtreg.SkippedException should be used instead of PassFailJFrame.forcePass(). > Affected tests: > 1. > https://github.com/openjdk/jdk/blob/master/test/jdk/java/awt/Desktop/BrowseTest.java > 2. > https://github.com/openjdk/jdk/blob/master/test/jdk/java/awt/Desktop/OpenTest.java > 3. > https://github.com/openjdk/jdk/blob/master/test/jdk/java/awt/Desktop/EditAndPrintTest/EditAndPrintTest.java > > Testing: > All the changed tests are tested manually and found to be working fine. Looks good to me. Thank you for updating these tests to throw `SkippedException` instead of pretending to pass. Update copyright years in all the updated files. ------------- Changes requested by aivanov (Reviewer). PR Review: https://git.openjdk.org/jdk/pull/24820#pullrequestreview-2788534540
