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

Reply via email to