On Fri, 6 Jun 2025 03:18:15 GMT, Prasanta Sadhukhan <psadhuk...@openjdk.org> wrote:
>> This is updated. (I have no preference on the behavior regarding >> UnsupportedLookAndFeelExceptions; this was copied and pasted from other test >> files.) > > I guess I am for not treating UnsupportedLookAndFeelException as an error. It > will cause the test to stop executing the next L&F if there is one in the > list after the unsupported L&F. Previous one is still better in my opinion > if we do not want to check for setLookAndFeel return.. > > I have seen this exception in CI testing for GTK L&F but I guess it's for > headless environment so I agree it's rare and will not happen in normal > case..I was just thinking why to cause unnecessary execution..It maynot > matter much for this test but for tests which uses Robot and has > delay/waitForIdle executing already run L&F because of UnsupportedException > will cause additional cycle. I updated this so a UnsupportedLookAndFeelException logs an error but does not abort the test. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/25443#discussion_r2131555608
