RFR: 8341453: java/awt/a11y/AccessibleJTableTest.java fails in some cases where the test tables are not visible

[Manukumar V S]

On Thu, 17 Oct 2024 18:53:53 GMT, Phil Race <prr at openjdk.org> wrote:

>> This issue can be fixed by setting the preferred scrollable viewport size of the two tables using
>>         JTable table = new JTable(data, columnNames);
>>     +  table.setPreferredScrollableViewportSize(table.getPreferredSize());
>>         JTable secondTable = new JTable(data, columnNames);
>>     +  secondTable.setPreferredScrollableViewportSize(secondTable.getPreferredSize());
>>     
>>     The other changes in imports section are just re-arrangements and expansion of imports.
>> 
>> Testing: This is a manual test and it is tested and verified manually in my local system
>> Screenshots of before and after the fix are attached in the bug.
>
> "+ "Using mouse drag the header of the last culumn so the last column becomes the first one.""
> 
> culumn -> column 
> 
> And would it be better to use PassFailJFrame for this test ?

@prrace Yes, it is better to use PassFailJFrame for this test and I have created a bug for the conversion https://bugs.openjdk.org/browse/JDK-8342663. I will work on this. I'm not mixing it with this fix as converting this test alone to use PassFailJFrame may not be feasible as all the tests inside the folder java/awt/a11y has a common abstract parent class AccessibleComponentTest, so it will be better to do the conversion for the whole set of tests inside java/awt/a11y folder together.

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

PR Comment: https://git.openjdk.org/jdk/pull/21549#issuecomment-2425724751