On Tue, 9 Sep 2025 20:41:06 GMT, Alexander Zvegintsev <[email protected]> wrote:
>> Prasanta Sadhukhan has updated the pull request incrementally with one >> additional commit since the last revision: >> >> Clarify instruction > > 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.. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/27162#discussion_r2335318761
