RFR: 8296275: Write a test to verify setAccelerator method of JMenuItem [v6]

Naveen Narayanan duke at openjdk.org
Fri Dec 9 05:26:40 UTC 2022


On Tue, 29 Nov 2022 16:04:27 GMT, Alexey Ivanov <aivanov at openjdk.org> wrote:

>> Naveen Narayanan has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   8296275: Review comments fixed.
>
> test/jdk/java/awt/Desktop/JMenuItemSetAcceleratorTest.java line 48:
> 
>> 46:     private static JFrame frame;
>> 47:     private static final CountDownLatch actionLatch = new CountDownLatch(1);
>> 48:     private static Robot robot;
> 
> My IDE issues a warning, the `robot` field can be converted to local variable. Indeed, it isn't used outside of `main`.

Since the test checks functionality of JMenuItem, its moved to JMenuItem folder.

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

PR: https://git.openjdk.org/jdk/pull/11035



More information about the client-libs-dev mailing list