On Thu, 9 Jan 2025 19:16:02 GMT, Alexey Ivanov <aiva...@openjdk.org> wrote:
>> **Problem:** >> >> The `javax/swing/JRadioButton/ButtonGroupFocus/ButtonGroupFocusTest.java` >> test fails in CI on Linux. >> >> The focus is on _Button 2_ instead of _Button 4_. >> >> **Root cause:** >> >> The additional logging revealed, an expected `KEY_PRESS` event. The test >> uses `Robot` API to press the <kbd>Tab</kbd> key and release it. When the >> test fails, there are two `KEY_PRESS` events followed by a single >> `KEY_RELEASE` event. Because the <kbd>Tab</kbd> key is pressed twice, the >> focus moves twice: from _Button 2_ (the initial state) to >> _Button 4_ and then back to _Button 2_. >> >> **Fix:** >> >> Use `CountDownLatch`es to track whether a radio button received focus. >> Detect the case where two `KEY_PRESS` events moved focus to _Button 2_ >> and report the failure. >> >> Log focus movements and dispatched key events to facilitate failure analysis. >> >> Take a screenshot of the test frame in case of failure. >> >> These CI hosts seem to be quite slow, removing the delay added by >> `robot.setAutoDelay(100)` has reduced the number of unexpected `KEY_PRESS` >> events. This didn't affect Windows or macOS. > > Alexey Ivanov has updated the pull request incrementally with one additional > commit since the last revision: > > Replace AWTEventListener and PropertyChangeListener with lambdas Marked as reviewed by dnguyen (Committer). ------------- PR Review: https://git.openjdk.org/jdk/pull/22977#pullrequestreview-2540740749
