RFR: 8327835: Convert java/awt/FileDialog/RegexpFilterTest/RegexpFilterTest applet test to main [v2]

Alexey Ivanov aivanov at openjdk.org
Tue Mar 12 09:40:15 UTC 2024


On Mon, 11 Mar 2024 23:18:28 GMT, Alexander Zvegintsev <azvegint at openjdk.org> wrote:

>> The `java/awt/FileDialog/RegexpFilterTest/RegexpFilterTest` test is written to test the very specific functionality of `XFileDialogPeer`, so it shouldn't be run on platforms other than Linux.
>> 
>> We also need to set `sun.awt.disableGtkFileDialogs` to `true` to be able to test `XFileDialogPeer`, because by default it tries to use the Gtk file dialog if it is available.
>> The Gtk file dialog has a search field hidden under the magnifying glass icon, but it has a different syntax.
>> 
>> The Windows system file dialog allows you to use wildcard filtering in the "File name" field by hitting the `enter` afterwards, but again, this test was written to test our own file dialog implementation, not the system's.
>
> Alexander Zvegintsev has updated the pull request incrementally with one additional commit since the last revision:
> 
>   better chaining

Changes requested by aivanov (Reviewer).

test/jdk/java/awt/FileDialog/RegexpFilterTest.java line 59:

> 57: 
> 58:     public static void main(String[] args) throws Exception {
> 59:         new PassFailJFrame.Builder()

Suggestion:

        PassFailJFrame.builder()

Please, use the helper method `builder()` instead of instantiating the `Builder` class explicitly.

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

PR Review: https://git.openjdk.org/jdk/pull/18203#pullrequestreview-1930546191
PR Review Comment: https://git.openjdk.org/jdk/pull/18203#discussion_r1521143484


More information about the client-libs-dev mailing list