RFR: 8368185: Test javax/swing/plaf/synth/SynthButtonUI/6276188/bug6276188.java failed: Synth ButtonUI does not handle PRESSED & MOUSE_OVER state [v2]
Prasanta Sadhukhan
psadhukhan at openjdk.org
Fri Oct 3 11:05:08 UTC 2025
On Fri, 3 Oct 2025 09:36:16 GMT, Alexey Ivanov <aivanov at openjdk.org> wrote:
>> Since the color change to Red will happen on MOUSE_PRESSED and MOUSE_OVER state, I wanted to give it more time to be in red color state so that robot can pick the color..
>> The failure was that it was picking up Green color even though the CI failure artifact shows background is Red
>
> Yes, I got this… A delay of 2 seconds on each run means the test will *always* wait for 2 seconds even if the update occurred in 0.1 seconds.
>
> It's fine as it is, yet I'd rather use a better synchronisation to allow the test to complete quicker, at the very least I'd investigate whether it's possible and whether the more complicated logic in the test gives a time bonus as the result.
Used CountDownLatch
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/27444#discussion_r2401534524
More information about the client-libs-dev
mailing list