On Tue, 9 Sep 2025 06:28:08 GMT, Alexander Zvegintsev <[email protected]> 
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.

This pull request has now been integrated.

Changeset: 3db3c062
Author:    Alexander Zvegintsev <[email protected]>
URL:       
https://git.openjdk.org/jdk/commit/3db3c06218f37ed4f14d4f53538663d2a5547095
Stats:     116 lines in 2 files changed: 39 ins; 24 del; 53 mod

8366852: java/awt/Choice/ChoiceMouseWheelTest/ChoiceMouseWheelTest.java test is 
failing

Reviewed-by: psadhukhan, honkar

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

PR: https://git.openjdk.org/jdk/pull/27161

Reply via email to