On Tue, 30 Aug 2022 10:44:39 GMT, Marius Hanl <mh...@openjdk.org> wrote:
>> There are no exceptions expected to happen in these new tests and I don't >> see that we use try-finally with `StageLoader` by default, is this change >> really required? > > What we usually do is to add a global variable like this: > `private StageLoader stageLoader;` > and in the junit after method something like this: > `if (stageLoader != null) stageLoader.dispose();` > > So tests are free to init and use the `stageLoader` and it will always call > `dispose()` after the test, no matter if it failed or not. > > cc @kleopatra > see for example: `TableCellTest` Thanks @Maran23, that is useful. ------------- PR: https://git.openjdk.org/jfx/pull/886