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