RFR: 8310024: Skip failing scene change tests on macOS

Kevin Rushforth kcr at openjdk.org
Wed Jun 14 16:02:04 UTC 2023


On Wed, 14 Jun 2023 15:35:40 GMT, Andy Goryachev <angorya at openjdk.org> wrote:

>> Simple fix to skip two failing tests on macOS until the underlying bugs are fixed:
>> 
>> SceneChangeEventsTest::testSceneChange -- [JDK-8300094](https://bugs.openjdk.org/browse/JDK-8300094)
>> SceneChangeShouldNotFocusStageTest::windowShouldRemainIconified -- [JDK-8305675](https://bugs.openjdk.org/browse/JDK-8305675)
>> 
>> With this patch, we now get a clean test run on macOS 12 and 13.
>
> tests/system/src/test/java/test/robot/javafx/scene/SceneChangeShouldNotFocusStageTest.java line 69:
> 
>> 67:     public static void exit() {
>> 68:         if (stage != null) {
>> 69:             Util.shutdown(stage);
> 
> I wonder if we should move the null check to Util.shutdown() to make things easier

Maybe. If so, we would also need a check to avoid calling shutdown if the platform was never started. In this specific case, I know that if the stage is null, then the platform was not started, but the utility method could not make that assumption.

In any case, I would want to do it as a follow-up RFE for the test utility and not part of a simple test exclusion bug fix.

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

PR Review Comment: https://git.openjdk.org/jfx/pull/1153#discussion_r1229854371


More information about the openjfx-dev mailing list