On Fri, 25 Apr 2025 19:52:06 GMT, Rajat Mahajan <rmaha...@openjdk.org> wrote:
>> Details: >> Refactored code as requested in the Bug description. >> >> Tested and verified the test passes. > > Rajat Mahajan has updated the pull request incrementally with five additional > commits since the last revision: > > - Update test/jdk/javax/swing/JRadioButton/8033699/bug8033699.java > > Co-authored-by: Alexey Ivanov <alexey.iva...@oracle.com> > - Update test/jdk/javax/swing/JRadioButton/8033699/bug8033699.java > > Co-authored-by: Alexey Ivanov <alexey.iva...@oracle.com> > - Update test/jdk/javax/swing/JRadioButton/8033699/bug8033699.java > > Co-authored-by: Alexey Ivanov <alexey.iva...@oracle.com> > - Update test/jdk/javax/swing/JRadioButton/8033699/bug8033699.java > > Co-authored-by: Alexey Ivanov <alexey.iva...@oracle.com> > - Update test/jdk/javax/swing/JRadioButton/8033699/bug8033699.java > > Co-authored-by: Alexey Ivanov <alexey.iva...@oracle.com> Changes requested by aivanov (Reviewer). test/jdk/javax/swing/JRadioButton/8033699/bug8033699.java line 62: > 60: robot = new Robot(); > 61: > 62: // Get all installed Look and Feels Huh, it was already there. Anyway. test/jdk/javax/swing/JRadioButton/8033699/bug8033699.java line 63: > 61: robot = new Robot(); > 62: SwingUtilities.invokeAndWait(() -> > 63: focusManager = > KeyboardFocusManager.getCurrentKeyboardFocusManager()); Suggestion: robot = new Robot(); SwingUtilities.invokeAndWait(() -> focusManager = KeyboardFocusManager.getCurrentKeyboardFocusManager()); test/jdk/javax/swing/JRadioButton/8033699/bug8033699.java line 120: > 118: runTest9(); > 119: robot.delay(100); > 120: } catch (Exception e) { Let's make the error message more informative: } catch (Exception e) { Throwable cause = e.getCause(); throw new RuntimeException("Error testing LaF: " + laf.getName() + (cause != null ? " - " + cause.getMessage() : ""), e); } finally { GitHub wouldn't allow add this as suggestion as these lines aren't modified. This produces the following output which is more descriptive since it also includes the real error message. Testing LaF: Metal Radio Button Group Go To Next Component through Tab Key failed Exception in thread "main" java.lang.RuntimeException: Error testing LaF: Metal - Focus is not on Radio Button Single as Expected at bug8033699.testLaF(bug8033699.java:125) at bug8033699.main(bug8033699.java:67) Caused by: java.lang.reflect.InvocationTargetException at … Caused by: java.lang.RuntimeException: Focus is not on Radio Button Single as Expected at bug8033699.lambda$runTest1$3(bug8033699.java:201) at … ------------- PR Review: https://git.openjdk.org/jdk/pull/24384#pullrequestreview-2795152173 PR Review Comment: https://git.openjdk.org/jdk/pull/24384#discussion_r2060808469 PR Review Comment: https://git.openjdk.org/jdk/pull/24384#discussion_r2060807687 PR Review Comment: https://git.openjdk.org/jdk/pull/24384#discussion_r2060815567