On Mon, 23 Oct 2023 03:28:00 GMT, Renjith Kannath Pariyangad <d...@openjdk.org> 
wrote:

>> Hi Reviewers,
>> 
>> I have added additional CountDownLatch for making sure button is visible 
>> before proceeding,(observe this was one of the key reason for failure) so 
>> this helps to improve test case stability. Modified _await()_ function by 
>> adding 1sec timeout with exception, this will help to reduce total execution 
>> time in case of failure. 
>> Please review and let me know your suggestions, if any.
>
> Renjith Kannath Pariyangad has updated the pull request incrementally with 
> one additional commit since the last revision:
> 
>   Added null check before frame dispose

The updated test failed once on Linux when I ran the test on Friday, the 
exception was:


Exception: java.lang.RuntimeException: Didn't receive focus in time


Today I also got a few failures in 8u:


Exception: java.lang.RuntimeException: Mouse is not pressed


on Linux and on Windows.

The good thing is that failure takes about 10 seconds instead of more than 5 
minutes.

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

PR Comment: https://git.openjdk.org/jdk/pull/16281#issuecomment-1775706452

Reply via email to