RFR: 8351884: Refactor bug8033699.java test code

[Harshitha Onkar]

On Wed, 2 Apr 2025 17:34:34 GMT, Rajat Mahajan <rmahajan at openjdk.org> wrote:

> Details:
> Refactored code as requested in the Bug description.
> 
> Tested and verified the test passes.

LGTM apart from minor inline suggestion.

Since the test has lot of tab movements, does CI testing look good on all platforms on multiple runs (repeat) ?

test/jdk/javax/swing/JRadioButton/8033699/bug8033699.java line 181:

> 179:         mainFrame.getContentPane()
> 180:                  .setLayout(new BoxLayout(mainFrame.getContentPane(),
> 181:                                           BoxLayout.Y_AXIS));

Minor nit:
Can be simplified by removing .getContentPane() from ln#170-173) as well similar to the following.

Suggestion:

        mainFrame.setLayout(new BoxLayout(mainFrame.getContentPane(),
                                          BoxLayout.Y_AXIS));

-------------

Marked as reviewed by honkar (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/24384#pullrequestreview-2769620853
PR Review Comment: https://git.openjdk.org/jdk/pull/24384#discussion_r2045355995