RFR: 8315484: java/awt/dnd/RejectDragDropActionTest.java timed out [v2]

Harshitha Onkar honkar at openjdk.org
Tue Oct 3 18:11:39 UTC 2023


On Tue, 3 Oct 2023 17:59:19 GMT, Damon Nguyen <dnguyen at openjdk.org> wrote:

>> test/jdk/java/awt/dnd/RejectDragDropActionTest.java line 90:
>> 
>>> 88:             Robot robot = new Robot();
>>> 89:             robot.waitForIdle();
>>> 90:             robot.delay(DELAY_TIME);
>> 
>> Does adding `robot.autoWaitForIdle(true)` additionally increase the stability of the test since it has subsequent DnD events performed by the robot?
>
> Sure, I just added it. Previously when I initially worked on this test, setAutoWaitForIdle had a weird interaction with how the test was setup. Since the test used a delay of 50ms between each of the movements when dragging the selection, it caused long delays. Looks like this is resolved now, and the method doesn't add any unreasonable delays anymore.

Sounds good!

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

PR Review Comment: https://git.openjdk.org/jdk/pull/16018#discussion_r1344525247


More information about the client-libs-dev mailing list