RFR: 8286172: Create an automated test for JDK-4516019 [v2]

[Srinivas Mandalika]

On Mon, 23 May 2022 17:20:38 GMT, Alisen Chung <achung at openjdk.org> wrote:

>> Srinivas Mandalika has updated the pull request incrementally with two additional commits since the last revision:
>> 
>>  - Review comments fixed: Fixed file path, removed redundant sleep call etc.
>>  - Review comments fixed: Removed file under the 'bug id' folder structure
>
> test/jdk/javax/swing/JSpinner/JSpinnerFocusTest.java line 69:
> 
>> 67:         try {
>> 68:             SwingUtilities.invokeAndWait(() -> createGUI());
>> 69:             Thread.sleep(1000);
> 
> Would it be better to make robot a global variable and use Robot.waitForIdle() here? Sleeping might slow down the test unnecessarily

Fixed as suggested.

> test/jdk/javax/swing/JSpinner/JSpinnerFocusTest.java line 71:
> 
>> 69:             Thread.sleep(1000);
>> 70:             runTest();
>> 71:             Thread.sleep(1000);
> 
> same here

Fixed as suggested.

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

PR: https://git.openjdk.java.net/jdk/pull/8546