RFR: 8339233: Test javax/swing/JButton/SwingButtonResizeTestWithOpenGL.java#id failed: Button renderings are different after window resize [v2]
Harshitha Onkar
honkar at openjdk.org
Tue Sep 3 16:44:19 UTC 2024
On Sat, 31 Aug 2024 09:08:59 GMT, Manukumar V S <mvs at openjdk.org> wrote:
>> This is a highly intermittent failure, and it failed only once in CI on a particular windows machine and passed on all other runs.
>>
>> Failure reason:
>> When the button was displayed for the first time, the focus rectangle drawn on the button text(Button A) was not seen(failure images attached in the bug).
>>
>> Fix:
>> To stabilize the test, I have added a line -> button.setFocusPainted(false), so that the focus rectangle will not be painted
>
> Manukumar V S has updated the pull request incrementally with one additional commit since the last revision:
>
> Review Comments Fixed : Further stabilized the test by increasing the delay as well as removing some unwanted lines of code
frame.setExtendedState(JFrame.MAXIMIZED_BOTH);
System.out.println("Frame is maximized");
robot.waitForIdle();
You missed adding delays after frame state changes, in the above case after robot.waitForIdle() adding robot.delay(100) can stabilize the test. This applies to all the places where frame.setExtendedState() is used.
-------------
Changes requested by honkar (Reviewer).
PR Review: https://git.openjdk.org/jdk/pull/20777#pullrequestreview-2277965584
More information about the client-libs-dev
mailing list