On Thu, 22 Oct 2020 14:22:32 GMT, Pankaj Bansal <pban...@openjdk.org> wrote:
> The test test.robot.javafx.scene.MouseLocationOnScreenTest fails > intermittently on Windows and Mac system though it fails on Windows more > often than Mac. > > The test uses robot to move the mouse at different positions on screen using > mouseMove and then validates the mouse coordinates by robot.getMouseX/Y. If > the coordinates returned from Robot.getMouseX/Y are same as mouse coordinates > passed to Robot.mouseMove, the test passes, else the test fails. > > The issue here is that there is no wait/delay between the calls to mouseMove > and getMouseX/Y. When Robot.mouseMove is used to move mouse to a coordinate, > robot in turn sends the event to native system. When Robot.getMouseX/Y is > called to get the current mouse coordinates, robot again uses native > functions to get the mouse location. In current test, it is possible that the > Robot.mouseMove is called with particular x,y coordinates and which in turn > has send the events to native. As there is no delay/wait after calling > Robot.mouseMove, When the Robot.getMouseX/Y is called, it is possible that > the earlier event sent to native has not been processed yet. So the mouse has > not been moved and Robot.getMouseX/Y may return older mouse coordinates. This > results in the test to fail. > > In short, the Robot.mouseMove is an asynchronous call, so it should be used > in same way. A follow up bug will be filled to update the specification to > reflect this. The test is assuming it to be synchronous and results in > intermittent failures. > > The fix is to add small delay after calling Robot.mouseMove. Along with this, > now as the test will take more time, so the timeout time for test is > increased to 120s. Also, I have divided the calls to edge(...) among more > Util.runAndWait calls as there is a timeout limit for Util.runAndWait which > is set to 10s. > > I tested this on Windows 10, Ubuntu 18.04, Ubuntu 20.04, Oracle Linux 8.2, > Mac 10.13. For me this test is passing in less than 50 seconds on all > platforms. Following command can be used to run the test > gradle --continue --info -PFULL_TEST=true -PUSE_ROBOT=true > :systemTest:cleanTest :systemTests:test --tests > test.robot.javafx.scene.MouseLocationOnScreenTest This pull request has now been integrated. Changeset: 2c675557 Author: Pankaj Bansal <pban...@openjdk.org> Committer: Kevin Rushforth <k...@openjdk.org> URL: https://git.openjdk.java.net/jfx/commit/2c675557 Stats: 19 lines in 1 file changed: 18 ins; 0 del; 1 mod 8213573: MouseLocationOnScreenTest fails intermittently Reviewed-by: kcr ------------- PR: https://git.openjdk.java.net/jfx/pull/333
