RFR: 8364363: Modify some manual test instructions [v2]

Damon Nguyen dnguyen at openjdk.org
Wed Aug 13 20:32:12 UTC 2025


On Wed, 6 Aug 2025 20:10:24 GMT, Alexey Ivanov <aivanov at openjdk.org> wrote:

>> Damon Nguyen has updated the pull request incrementally with two additional commits since the last revision:
>> 
>>  - Fix correcting character limit to undo all changes and leave file untouched.
>>  - Convert test to auto. Revert TextArea test changes
>
> test/jdk/java/awt/List/MouseDraggedOriginatedByScrollBarTest.java line 48:
> 
>> 46:             3) The test fails if you see messages about drag
>> 47:             events for anything other than list0.
>> 48:             The test passes if you don't.
> 
> Is it possible to automate this part? I mean, automatically flag the test as failed (after displaying a message to the tester after dragging is complete).

I have gone ahead and updated the test to be automated. I tested this on macOS (where the test skips since the test only works for XToolkit) and on Ubuntu on Xorg (where the test is automated and passes with no outputted events).

> test/jdk/java/awt/TextArea/TextAreaAppendScrollTest2.java line 71:
> 
>> 69:             coolStr.append("cool ");
>> 70:         }
>> 71:         area.append(coolStr.toString());
> 
> Is adding `12 * 15` _cools_ not enough?
> 
> The original instructions are pretty clear: if you see the exclamation marks, then auto-scrolling works correctly; otherwise, it auto-scrolling doesn't work as expected and the tester has to click **Fail**.

@aivanov-jdk Is problem-listing this test the ideal solution?

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

PR Review Comment: https://git.openjdk.org/jdk/pull/26636#discussion_r2274553279
PR Review Comment: https://git.openjdk.org/jdk/pull/26636#discussion_r2274555264


More information about the client-libs-dev mailing list