On Sun, 16 Mar 2025 08:56:24 GMT, Manukumar V S <m...@openjdk.org> wrote:
> **Issue** > java/awt/Desktop/MailTest.java fails in platforms where Action.MAIL is not > supported > > **Reason** > In `MailTest.java`, there is a condition check(`if > (!desktop.isSupported(Desktop.Action.MAIL))`) which will force pass the test > if the corresponding 'Action' is not supported by the platform. But, > apparently, this is not working good and the code flow went past this and > fails in desktop.mail() method with an UnsupportedOperationException. > > **Fix** > Even though we are calling `PassFailJFrame.forcePass()` if the 'Action.MAIL' > is unsupported, the PassFailJFrame just count downs a latch and the actual > action will be taken later only(in `awaitAndCheck()`). But at the meantime, > the desktop.mail() call gets executed in the constructor of MailTest() and it > will result in an UnsupportedOperationException. So, the fix is to return > from the constructor immediately if the operation is 'unsupported'. > > **Testing** > This is a manual test, so it is tested locally and found to be working fine. This pull request has now been integrated. Changeset: c7f33388 Author: Manukumar V S <m...@openjdk.org> Committer: Alexey Ivanov <aiva...@openjdk.org> URL: https://git.openjdk.org/jdk/commit/c7f333888be052aa37fe878bfc2785fc47fbeaaa Stats: 37 lines in 1 file changed: 17 ins; 17 del; 3 mod 8352109: java/awt/Desktop/MailTest.java fails in platforms where Action.MAIL is not supported Reviewed-by: aivanov ------------- PR: https://git.openjdk.org/jdk/pull/24072
