RFR: 8315655: [macos] Right click and dragging over a component with a popup menu will open the popup [v6]
Damon Nguyen
dnguyen at openjdk.org
Tue Jun 11 20:53:51 UTC 2024
On Sat, 8 Jun 2024 00:49:29 GMT, Alisen Chung <achung at openjdk.org> wrote:
>> test/jdk/javax/swing/JPopupMenu/MouseDragPopupTest.java line 59:
>>
>>> 57: });
>>> 58: robot.waitForIdle();
>>> 59: robot.delay(1000);
>>
>> Suggestion:
>>
>> robot.setAutoDelay(100);
>> robot.setAutoWaitForIdle(true);
>>
>> SwingUtilities.invokeAndWait(() -> {
>> createAndShowGUI();
>> });
>> robot.delay(1000);
>>
>>
>> I was able to remove all the individual `waitForIdle()` calls and replaced it with `setAutoWaitForIdle()`. Still seemed to work fine on macOS.
>
> Most tests i've looked at aren't using setAutoWaitForIdle(). Are there any issues with test stability?
I believe you're right. I recall mouse drag movements having some oddity with setAutoWaitForIdle. It's probably fine as is.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/19569#discussion_r1635466752
More information about the client-libs-dev
mailing list