On Wed, 14 Feb 2024 21:01:59 GMT, Andy Goryachev <ango...@openjdk.org> wrote:

>> It is the count, how often update item was called. 
>> That it got smaller shows, that we call it less often, as the item was not 
>> changed (which is a good thing).
>> That also matches with your observation, which is the expected output here.
>> 
>> I saw that this was changed for some tests in 
>> https://github.com/openjdk/jfx/pull/863, we also can do something similar 
>> here as well.
>
> on second thought, this is ok as is - I asked because the code is not testing 
> a specific requirement.
> The second `assertEquals(18)` might be testing a specific requirement - that 
> the count has not changed, right?

Yes, as far as I can see.

-------------

PR Review Comment: https://git.openjdk.org/jfx/pull/1360#discussion_r1490914400

Reply via email to