On Tue, 27 Apr 2021 16:30:52 GMT, mstr2 <github.com+43553916+ms...@openjdk.org> 
wrote:

> I think you're right that the same test could also be added for 
> TreeTableView, and that the test should probably cover the entirety of the 
> change notification. However, my preference is to not create a test that 
> duplicates all aspects of the specific program that was attached to the JBS 
> issue. I think it's more meaningful to test the most minimal setup that can 
> reproduce the underlying issue; adding bells and whistles (as was done in the 
> JBS code sample) feels like adding unnecessary noise to the test.

Arguable .. we might miss some aspect of the setup in the report when not 
testing explicitly for it :)

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

PR: https://git.openjdk.java.net/jfx/pull/480

Reply via email to