RFR: 8354233: Open some JTable bugs 6 [v2]
Alisen Chung
achung at openjdk.org
Mon Apr 21 23:24:32 UTC 2025
On Mon, 21 Apr 2025 20:12:43 GMT, Alexander Zuev <kizune at openjdk.org> wrote:
>> Alisen Chung has updated the pull request incrementally with one additional commit since the last revision:
>>
>> update jtreg paths, volatile, manual jtreg tag
>
> test/jdk/javax/swing/JTable/bug4460382.java line 120:
>
>> 118: row1 = editingRow;
>> 119: setValueAt(value, editingRow, editingColumn);
>> 120: row2 = editingRow;
>
> Can you tell me what are we testing here? We extending JTable and we are checking that during the cell content replacement some random internal protected variable does not change. What does it tells us? Why is it a criteria of test passing or not?
i think this test should probably be removed, since the manual test was also like this in that it didn't really test anything.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/24735#discussion_r2053098563
More information about the client-libs-dev
mailing list