On Fri, 28 Apr 2023 13:04:44 GMT, Kevin Rushforth <[email protected]> wrote:

>> The test was failing on the first run and then it was not failing. Cause for 
>> the failure looked to be the UI state not getting updated. Hence added call 
>> to `Toolkit` `firePulse` method.
>> 
>> Able to run the test consistently without failure in Linux after the fix.
>
> tests/system/src/test/java/test/robot/javafx/scene/SliderTooltipNPETest.java 
> line 97:
> 
>> 95:                             (int)(scene.getWindow().getY() + 
>> scene.getY() +
>> 96:                                 slider.getLayoutY() + 
>> slider.getLayoutBounds().getHeight()/2));
>> 97:             Toolkit.getToolkit().firePulse();
> 
> A system test should not need to use `firePulse`, and I can't think why this 
> would help. In any case, I still see the failure on my Ubuntu 22.04 VM.

Oh Okay! Let me check on that again.
Looks like the failure is not consistent in all the Linux systems. Even without 
the fix it was failing only on the first run for me in VM as well as in Ubuntu 
workstation.

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

PR Review Comment: https://git.openjdk.org/jfx/pull/1119#discussion_r1180520125

Reply via email to