On Tue, 26 May 2020 11:12:25 GMT, Ambarish Rapte <ara...@openjdk.org> wrote:
> Copied from JBS: > As a best practice, unit test classes that are run by Junit should not extend > javafx.application.Application. > JUnit constructs an instance of a test class for each test method that it > runs. This will be different from the > instance of the object that is constructed when Application.launch method is > called, and this can lead to unexpected > behavior. Most of our tests use helper classes or nested (static) sub classes > of Application, but the following two > test classes do not: > tests/system/src/test/java/test/com/sun/javafx/image/impl/ImageRaceTest.java > tests/system/src/test/java/test/javafx/scene/control/AccordionTitlePaneLeakTest.java > > ---- > Changes in the tests are: > ImageRaceTest.java-> Does not require a sub class of Application. > AccordionTitlePaneLeakTest.java-> Required minor changes to create a static > sub class. Marked as reviewed by kcr (Lead). ------------- PR: https://git.openjdk.java.net/jfx/pull/237
