On Tue, 13 Feb 2024 21:34:09 GMT, Alisen Chung <ach...@openjdk.org> wrote:

> Root cause of the test failure was fixed with 
> https://bugs.openjdk.org/browse/JDK-8316931, updating this test since the 
> other fix also included a test update.

test/jdk/java/awt/TrayIcon/DisposeInActionEventTest/DisposeInActionEventTest.java
 line 51:

> 49:         String instructions;
> 50:         traySupported = SystemTray.isSupported();
> 51:         if (!traySupported) {

`SystemTray.isSupported()` can be directly evaluated in `if` condition.  
`traySupported` variable can be removed.

test/jdk/java/awt/TrayIcon/DisposeInActionEventTest/DisposeInActionEventTest.java
 line 53:

> 51:         if (!traySupported) {
> 52:             instructions = "The test cannot be run because SystemTray is 
> not supported.\n" +
> 53:                            "Simply press PASS button.";

If systemTray is not supported, is it a good idea to just return and pass or 
skip the test ?

test/jdk/java/awt/TrayIcon/DisposeInActionEventTest/DisposeInActionEventTest.java
 line 62:

> 60:             }
> 61: 
> 62:             instructions = "When the test starts, it adds the icon to the 
> tray aread. If you\n" +

Suggestion:

            instructions = "When the test starts, it adds the icon to the tray 
area. If you\n" +

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

PR Review Comment: https://git.openjdk.org/jdk/pull/17838#discussion_r1491355804
PR Review Comment: https://git.openjdk.org/jdk/pull/17838#discussion_r1491364867
PR Review Comment: https://git.openjdk.org/jdk/pull/17838#discussion_r1491368567

Reply via email to