On Fri, 10 Dec 2021 12:33:29 GMT, Johan Vos <j...@openjdk.org> wrote:
>> eduardsdv has updated the pull request incrementally with one additional >> commit since the last revision: >> >> 8276170: Add junit for VirtualFlow.scrollToTop(int) > > modules/javafx.controls/src/test/java/test/javafx/scene/control/ListViewKeyInputTest.java > line 1844: > >> 1842: } >> 1843: listView.setPrefHeight(130); // roughly room for four rows >> 1844: Toolkit.getToolkit().firePulse(); > > Why is this needed here? Without this line the test fails in the line 1865 with the message 'expected:<7> but was:<8>'. I think this is because in the line 1850 'listView.scrollTo(99)' is executed, which now does not set the position to 1. ------------- PR: https://git.openjdk.java.net/jfx/pull/656
