On Fri, 3 Dec 2021 18:47:24 GMT, Kevin Rushforth <k...@openjdk.org> wrote:
> The fix and new test looks fine. I added one comment/question regarding the > existing tests that check a count, but I'm OK with either answer. The suggestion to test for positive values of the counters is a good idea. The lower this counter the better, but if the counter is 0, this means there has been no evaluation of updateItem. This should cause other tests to fail, but it helps if it would trigger a failure in this test as well. I added these asserts on the places where previously the expected value was not 0. ------------- PR: https://git.openjdk.java.net/jfx/pull/683
