On Wed, 18 Jun 2025 10:43:38 GMT, Marius Hanl <mh...@openjdk.org> wrote:
>> modules/javafx.controls/src/test/java/test/com/sun/javafx/scene/control/infrastructure/VirtualFlowTestUtils.java
>> line 349:
>>
>>> 347: flow = (VirtualFlow<?>)control.lookup("#virtual-flow");
>>> 348:
>>> 349: if (sl != null) {
>>
>> should we have a check here with a helpful message (which recommends to use
>> `StageLoader`), instead of returning `null`?
>
> I thought about this as well. No strong opinion from my side.
Depends on the amount of noise it will produce during the test run but in
general i would also prefer some kind of message here.
-------------
PR Review Comment: https://git.openjdk.org/jfx/pull/1829#discussion_r2198083367
