On Mon, 16 Jan 2023 13:04:27 GMT, Tejesh R <[email protected]> wrote: >> In Aqua look and feel, `getDefaultButton()` of `JFileChooser` returns null. >> The fix overrides the method from `FileChooserUI` to return the >> `ApproveButton` of `FileChooser` which is similar to other L&F. >> The test is run for multiple iterations in CI machine. > > Tejesh R has updated the pull request incrementally with one additional > commit since the last revision: > > Updated based on review comments
> > This case is entirely covered by the test in #11901 for > > [JDK-8299522](https://bugs.openjdk.org/browse/JDK-8299522), which also > > ensures the button text isn't empty. > > Adding 8300084 to the `@bug` tag in that test is enough. It was my > > [original > > suggestion](https://github.com/openjdk/jdk/pull/11901#discussion_r1068659255): > > > Now you can fix this bug so that Aqua L&F doesn't return null and add > > > that bugid to this test. > > Yeah, but since #11901 handles only Approve button size except for Aqua L&F > and this fix is only for Aqua L&F I thought it would be better to not to > maintain any dependency between the test/bug (Though this bug was initiated > as part of #11901 fix). I hope maintaining simple test case for two bug would > be fine. It's exactly the point: **include Aqua L&F in #11901**. The Aqua L&F is excluded only because `getDefaultButton` returns null, isn't it? This fix makes it return non-null, therefore Aqua L&F can be included in testing. This fix should be integrated before [JDK-8299522](https://bugs.openjdk.org/browse/JDK-8299522). I will not approve #11901 until Aqua L&F is part of the `CustomApproveButtonTest.java` test. ------------- PR: https://git.openjdk.org/jdk/pull/12008
