RFR: 8366369: Add @requires linux for GTK L&F tests
Alexey Ivanov
aivanov at openjdk.org
Fri Aug 29 08:32:43 UTC 2025
On Fri, 29 Aug 2025 02:30:36 GMT, Prasanta Sadhukhan <psadhukhan at openjdk.org> wrote:
>> * Add `@requires os.family == "linux"` to prevent GTK L&F tests from running on other OS but Linux.
>> * Let any exceptions from `UIManager.setLookAndFeel` propagate.
>> * Remove `@author` tag.
>> * Expand imports in `test/jdk/com/sun/java/swing/plaf/gtk/Test6635110.java`.
>> * Add leading asterisks to jtreg comment block.
>
> test/jdk/com/sun/java/swing/plaf/gtk/4928019/bug4928019.java line 37:
>
>> 35: public class bug4928019 {
>> 36: public static void main(String[] args) throws Throwable {
>> 37: UIManager.setLookAndFeel("com.sun.java.swing.plaf.gtk.GTKLookAndFeel");
>
> WHy to restrict it to only GTK? We can iterate it for all installed L&Fs, no?
> We have similar fixes in particular L&F but then extend the test for all L&F just to ensure it works ok for all L&F..
This is the question to the original author of the test. I do not change the logic of the test. If the test is designed to run with GTK L&F only, let it be.
If you feel a test has to be extended to cover all L&Fs, submit a bug to expand the test to other L&Fs.
The purpose of this PR is to avoid running GTK L&F specific tests on the platforms where GTK L&F isn't available.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/26995#discussion_r2309536374
More information about the client-libs-dev
mailing list