RFR: 8225131: Test DragSourceMotionListenerTest.java fails on Windows [v2]
Tejesh R
tr at openjdk.org
Thu Sep 18 06:59:57 UTC 2025
On Tue, 16 Sep 2025 11:01:15 GMT, Alexey Ivanov <aivanov at openjdk.org> wrote:
>> Tejesh R has updated the pull request incrementally with one additional commit since the last revision:
>>
>> Review fix
>
> test/jdk/java/awt/dnd/DragSourceMotionListenerTest.java line 187:
>
>> 185: }
>> 186: robot.waitForIdle();
>> 187: robot.delay(100);
>
> Will the test be as stable without these two? Dragging isn't stopped here.
>
> In other words, isn't adding `waitForIdle` after dragging is fully finished at lines 198–199 but before releasing the mouse button and the <kbd>Ctrl</kbd> key.
Adding waitForIdle and delay at line 176 makes the test stable. I have removed all the changes and test passes multiple runs at CI without failure.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/27283#discussion_r2357731776
More information about the client-libs-dev
mailing list