On Fri, 11 Apr 2025 09:37:12 GMT, Gopal Pattnaik <d...@openjdk.org> wrote:
> There was a Assertion fail issue in mouse location test case JDK-8296554, > Reason: We felt the one mili second delay time for the Robot test may be > insufficient in few OS. > Solution: We Changed the delay time to three mili second. > > Verification: > Tested in Windows 11, and the Assert fail issue is not found. tests/system/src/test/java/test/robot/javafx/scene/MouseLocationOnScreenTest.java line 46: > 44: static CountDownLatch startupLatch = new CountDownLatch(1); > 45: static Robot robot; > 46: private static int DELAY_TIME = 3; minor: `Util.sleep()` could be moved to `validate()` question: even though the value is 3x the original, should it be longer to account for any hiccups? maybe 10? what do you think? ------------- PR Review Comment: https://git.openjdk.org/jfx/pull/1772#discussion_r2039703236
