RFR: 8277853: With Touch enabled devices scrollbar disappears and the table is scrolled to the beginning [v3]

[Kevin Rushforth]

On Tue, 25 Jan 2022 15:19:44 GMT, Marius Hanl <mhanl at openjdk.org> wrote:

>> meghanEmbrace has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Replaced the null test with an assert.
>
> modules/javafx.controls/src/test/java/test/javafx/scene/control/TableViewTest.java line 801:
> 
>> 799:      * Tests for specific bugs                                           *
>> 800:      ********************************************************************/
>> 801:     @Test public void test_jdk_8277853() {
> 
> Not sure if there is some preference around here but I always like if the test method is not just named after the ticket but instead what it actually tests, e.g. something like `testInvisibleScrollbarDoesNotScrollTableToBeginning`. Optionally the ticket number can be referenced in the javadoc then. But that's just me (just a side note here).

I almost made the same comment as you did. In general we no longer use the pattern of putting the bug ID in the test name. In this case all of the surrounding tests do that, so I decided not to mention it, but I am perfectly happy for the test name to be changed to reflect its function (with the bug ID in a comment).

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

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