On Tue, 27 Feb 2024 11:01:50 GMT, Alexey Ivanov <[email protected]> wrote:

>> Alisen Chung has updated the pull request incrementally with one additional 
>> commit since the last revision:
>> 
>>   remove system property set
>
> test/jdk/java/awt/TrayIcon/DisposeInActionEventTest/DisposeInActionEventTest.java
>  line 67:
> 
>> 65:                        "  something is wrong the corresponding message 
>> is displayed below.\n" +
>> 66:                        "  Repeat clicks several times. If no 'Test 
>> FAILED' messages\n" +
>> 67:                        "  are printed, press PASS button else FAIL 
>> button.";
> 
> It's not displayed below, it's to the right, isn't it?
> 
> Fail the test automatically if you can detect the failure. You may show a 
> message to the tester about the failure before shutting down the test.

This comment also implied amending the instructions to correspond to what the 
test does.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/17838#discussion_r1516304874

Reply via email to