On Wed, 10 Sep 2025 02:25:00 GMT, Prasanta Sadhukhan <[email protected]> 
wrote:

>> test/jdk/java/awt/event/MouseEvent/AltGraphModifierTest/AltGraphModifierTest.java
>>  line 52:
>> 
>>> 50:                     Navigate to
>>> 51:                       System Settings-> Keyboard-> Special Character 
>>> Entry
>>> 52:                     Select "Right Alt" option for the "Alternative 
>>> Characters Key"
>> 
>> Suggestion:
>> 
>>                     Select "Right Alt" option for the "Alternate Characters 
>> Key"
>> 
>> 
>> I guess we should add instructions for a test user to turn the setting back 
>> to default, or skip the test if their system does not have such a setting.
>> 
>> We also use the `forcePass/forceFail` approach:
>> * The test user may not have a chance to read the instructions all the way 
>> through if they are just following them.
>> * If the user accidentally presses a mouse button with the wrong or missing 
>> modifier, there is no way to retry.
>> 
>> So maybe we should add `logArea` and print theresults there, to allow the 
>> test user to make several attempts, and read the instructions till the end.
>
> Usually user should read the instructions till the end and then only attempt 
> the test, which is true for all the manual tests 
> but I got the point.
> I have added logArea and let user press Pass/Fail depending on the result 
> printed there..

What do you think about the following?

> I guess we should add instructions for a test user to turn the setting back 
> to default, or skip the test if their system does not have such a setting.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/27162#discussion_r2335924372

Reply via email to