RFR: 8366852: java/awt/Choice/ChoiceMouseWheelTest/ChoiceMouseWheelTest.java test is failing [v2]
Prasanta Sadhukhan
psadhukhan at openjdk.org
Fri Sep 12 03:50:23 UTC 2025
On Thu, 11 Sep 2025 14:51:27 GMT, Alexander Zvegintsev <azvegint at openjdk.org> wrote:
>> There are several major issues with the test:
>>
>> 1. The `frameExited` variable is set to true when the mouse pointer exits the frame. The test does not expect this to happen with the first mouse click on the `choice` when the mouse pointer is already over the test frame (frame mouseExited - > choice mouseEntered). A reset for the frameExited was added.
>> 2. The test does not wait for the frame to appear after the call to `setVisible()`, so the test can start operating not on a fully visible window (e.g. one with an appearance animation). A delay has been added.
>> 3. In macOS, the direction of rotation of the mouse wheel is different, which was not taken into account.
>>
>> Testing looks good.
>
> Alexander Zvegintsev has updated the pull request incrementally with one additional commit since the last revision:
>
> review comments
Marked as reviewed by psadhukhan (Reviewer).
-------------
PR Review: https://git.openjdk.org/jdk/pull/27161#pullrequestreview-3214489752
More information about the client-libs-dev
mailing list