RFR: 8306996: Open source Swing MenuItem related tests [v3]
Prasanta Sadhukhan
psadhukhan at openjdk.org
Thu May 11 09:23:45 UTC 2023
On Thu, 11 May 2023 08:45:09 GMT, Abhishek Kumar <abhiscxk at openjdk.org> wrote:
>> test/jdk/javax/swing/JMenuItem/bug4966168.java line 26:
>>
>>> 24: /* @test
>>> 25: @bug 4966168
>>> 26: @summary JInternalFrame not serializable in Motif & GTK L&F
>>
>> I dont see the test being tested for Motif & GTK which are not default L&F...Probably we should iterate for all L&F just to be sure..
>> Also, all `throw new Error` should ideally be changed to `throw new RuntimeException` for all tests in this PR..
>
> After modifying the test to run in all L&F, CI failure observed for linux and linux-aarch machine with the exception
> `javax.swing.UnsupportedLookAndFeelException: [GTK look and feel - com.sun.java.swing.plaf.gtk.GTKLookAndFeel] not supported on this platform`.
>
> On my local machine (Oracle Linux) test passed for all LAFs.
It probably needs headful tag for that.
Also, you should `continue` for UnsupportedLookAndFeelException...and not fail, just print a message and continue to next L&F
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/13822#discussion_r1190886666
More information about the client-libs-dev
mailing list