On Mon, 12 May 2025 13:40:06 GMT, Anass Baya <ab...@openjdk.org> wrote:
>However, the CountDownLatch is currently set to 1, which means the test only
>waits for the first open/close interaction to complete. As a result, it does
>not wait for the second attempt (opening the dialog again and pressing ESC to
>close it), because the latch reaches zero after the first attempt.
This causes the test to proceed immediately to the validation step:
if (fd.isVisible()) {
throw new RuntimeException("File Dialog is not closed");
I don't see the existing code ignores the second attempt, that is due to longer
delay put after the robot operations.
You can add print statements and observe the behavior.
However, I agree the countdown latch should be initialize to 2 to behave
correctly.
}
Other enhancements look good as threads are not required to verify the test.
-------------
PR Comment: https://git.openjdk.org/jdk/pull/25184#issuecomment-2883052617
PR Comment: https://git.openjdk.org/jdk/pull/25184#issuecomment-2883059066
