On Thu, 21 May 2020 10:05:25 GMT, Ajit Ghaisas <aghai...@openjdk.org> wrote:

>> It's a task to support cross-control/skin testing for recurring issues (like 
>> memory leaks on switching skins)
>> 
>> Basically, there's a utility class
>> - to access lists of all control/classes,
>> - to access/create all behaviors
>> - has alternative skins classes for all controls and utility to 
>> install/replace skins
>> 
>> POC:
>> - has test examples (not run in normal testing due to not following naming 
>> conventions) for using
>> - changed SkinDisposeContractTest to use
>
> modules/javafx.controls/src/test/java/test/com/sun/javafx/scene/control/infrastructure/ControlBehaviorTestExample.java
> line 52:
>> 51: @RunWith(Parameterized.class)
>> 52: public class ControlBehaviorTestExample {
>> 53:
> 
> This test does not execute when run as part of :controls:test. We need to 
> rename this class (and file) name to end with
> 'Test'. I tried doing it and lot of failures are seen.

that's the point - they are examples of how-to use it and see our tasks :)

Will address your other comments in a minute - and add two real tests (on 
skin/behavior memory leaks with the
misbehaving classes removed: then we'll have both the passing classes and 
informal ignores on what's going wrong.

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

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

Reply via email to