On Fri, 14 Nov 2025 09:37:29 GMT, Marius Hanl <[email protected]> wrote:
>> Ziad El Midaoui has updated the pull request incrementally with one >> additional commit since the last revision: >> >> Add test > > modules/javafx.controls/src/test/java/test/javafx/scene/control/TreeTableViewTest.java > line 6481: > >> 6479: >> 6480: TreeTableRow<String> rowAfter = findRow(table, "item B"); >> 6481: assertNotNull(rowAfter, "item B row should not be null"); > > Null check is not needed, since we do test the disclosure node position > instead The null check is just for safety that the item exist, not that much important. ------------- PR Review Comment: https://git.openjdk.org/jfx/pull/1971#discussion_r2527711938
