On Fri, 10 May 2024 05:06:53 GMT, Harshitha Onkar <[email protected]> wrote:

>> Tejesh R has updated the pull request incrementally with one additional 
>> commit since the last revision:
>> 
>>   Review fix + Eanbled column selection mode
>
> test/jdk/javax/swing/JTable/KeyBoardNavigation.java line 182:
> 
>> 180:                 2. Check all combinations of navigational keys in all 
>> four modes
>> 181:                   shift and control verifying each change to the 
>> selection against
>> 182:                   the spec. If it does, press "pass", otherwise press 
>> "fail".
> 
> "...in all four modes shift and control verifying each change to the 
> selection against the spec." 
> Can this sentence be rephrased for clarity ?

Rephrased.

> test/jdk/javax/swing/JTable/KeyBoardNavigation.java line 191:
> 
>> 189:                 .rows(11)
>> 190:                 .columns(50)
>> 191:                 .testUI(KeyBoardNavigation::initTest)
> 
> Default timeout of 5 mins is less to go through the test instructions. You 
> might want to increase by adding .testTimeOut().

Updated to 10min timeout.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/18855#discussion_r1596273490
PR Review Comment: https://git.openjdk.org/jdk/pull/18855#discussion_r1596273858

Reply via email to