RFR: 8328384: Convert java/awt/FileDialog/FileDialogOpenDirTest test to main [v4]
Alexander Zvegintsev
azvegint at openjdk.org
Thu Mar 21 16:38:22 UTC 2024
On Wed, 20 Mar 2024 15:48:41 GMT, Alexander Zuev <kizune at openjdk.org> wrote:
>> Convert test to main manual;
>> Move test to appropriate folder;
>> Delete old files;
>
> Alexander Zuev has updated the pull request incrementally with one additional commit since the last revision:
>
> Moving test metadata closer to the class definition for better readability;
> Making test automatically skip execution on unsuported toolkit;
> Eliminated the separate frame, moving button inside the PassFailJFrame;
> Corrected test instructions accordingly;
Marked as reviewed by azvegint (Reviewer).
test/jdk/java/awt/FileDialog/FileDialogOpenDirTest.java line 49:
> 47: JButton open = new JButton("Open File Dialog");
> 48: open.addActionListener(e -> {
> 49: new FileDialog((Frame) null).show();
Suggestion:
new FileDialog((Frame) null).setVisible(true);
show is deprecated
test/jdk/java/awt/FileDialog/FileDialogOpenDirTest.java line 68:
> 66: if (!toolkit.equals("sun.awt.X11.XToolkit")) {
> 67: throw new SkippedException("Test is not designed for toolkit " + toolkit);
> 68: }
Cay be moved to the beginning of the main, to bail out early.
-------------
PR Review: https://git.openjdk.org/jdk/pull/18361#pullrequestreview-1952775158
PR Review Comment: https://git.openjdk.org/jdk/pull/18361#discussion_r1534257573
PR Review Comment: https://git.openjdk.org/jdk/pull/18361#discussion_r1534258864
More information about the client-libs-dev
mailing list