RFR: 8307311: Timeouts on one macOS 12.6.1 host of two Swing JTableHeader tests

Alexander Zvegintsev azvegint at openjdk.org
Fri May 19 11:46:50 UTC 2023


On Fri, 19 May 2023 11:27:16 GMT, Prasanta Sadhukhan <psadhukhan at openjdk.org> wrote:

>> test/jdk/javax/swing/JTableHeader/6889007/bug6889007.java line 87:
>> 
>>> 85:             for(int i = -shift; i < width + 2*shift; i++) {
>>> 86:                 robot.mouseMove(x++, y);
>>> 87:                 robot.delay(100);
>> 
>> It looks like we don't need this delay since there is `robot.setAutoDelay(100);` on line 56. A total delay of 200ms seems too much to me.
>
> I dont think it's too much as compared to 500ms we have in some tests..Since I have tested in all platforms with this values without any problems observed, I will like to keep this value for now..

This seems OK to me for a rarely repeated operation.
Here it is repeated 405 times(at least for my testing), which leads to an additional ~40.5 s of the test execution time.
100ms is still a pretty solid delay for mouse movement.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/14026#discussion_r1198868406



More information about the client-libs-dev mailing list