On Wed, 12 May 2021 11:53:16 GMT, Ambarish Rapte <ara...@openjdk.org> wrote:

> The test fails intermittently on github build. But I do not observe this 
> issue locally on my Ubuntu20.04 VM.
> I am suspecting that the reason of failure is that the test are not cleaned 
> up correctly. The changes in this PR cleans up the test, by removing all 
> children added to root and hiding the stage.

modules/javafx.graphics/src/test/java/test/javafx/css/HonorDeveloperSettingsTest.java
 line 61:

> 59:     private Text text;
> 60:     Stage stage;
> 61:     Group root;

Can these be private?

modules/javafx.graphics/src/test/java/test/javafx/css/HonorDeveloperSettingsTest.java
 line 76:

> 74:         resetStyleManager();
> 75:         root.getChildren().clear();
> 76:         stage.hide();

I recommend protecting these with null checks. You might also set `stage` and 
`root` to null after clearing / hiding (but since they are instance variables 
that doesn't really matter).

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

PR: https://git.openjdk.java.net/jfx/pull/496

Reply via email to