On Sat, 26 Sep 2020 13:51:10 GMT, Kevin Rushforth <k...@openjdk.org> wrote:
>> Leon Linhart has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> Reverted incorrect change and improved test coverage
>
> modules/javafx.base/src/test/java/test/javafx/collections/ObservableListTest.java
> line 268:
>
>> 266:
>> 267: r = list.setAll();
>> 268: assertTrue(r);
>
> Can you also test that calling `setAll` when the list is currently empty
> returns true? Repeating one of the earlier
> checks should work:
> r = list.setAll("one");
> assertTrue(r);
Sure. Added in #284 5de1a51fd3a8d744acf0b831ab4f5a0441e417d7.
-------------
PR: https://git.openjdk.java.net/jfx/pull/284
