On Mon, 10 Feb 2025 18:41:07 GMT, Andy Goryachev <ango...@openjdk.org> wrote:
>> tests/system/src/test/java/test/robot/javafx/scene/NodeInitializationStressTest.java >> line 186: >> >>> 184: accessChart(c); >>> 185: c.getData().setAll(createNumberSeries()); >>> 186: c.getData().setAll(createNumberSeries()); >> >> Just curious - why do we do setAll twice in all the tests? > > good question! this code exercises both remove and add code paths. Maybe add a comment then? ------------- PR Review Comment: https://git.openjdk.org/jfx/pull/1697#discussion_r1966536274