On Tue, 16 Jan 2024 12:47:50 GMT, Alexey Ivanov <aiva...@openjdk.org> wrote:
> In addition to this, I propose removing `robot.delay(50);` from the > for-loops, or at least reducing this value, let's say, to 5. The test is unstable without a delay inside the for-loop. `robot.delay(5)` does the trick. The test passes successfully with 50 repeats. The above changes make the test significantly quicker. - Original version takes **25 minutes** for 50 repeats; - Yesterday's version — 30 minutes; - Today's version — 22 minutes; - With `delay(5)` — **7 minutes**; - ~~No delay — 5 minutes (*unstable*)~~. Thus, the modified version with `robot.delay(5)` instead of `robot.delay(50)` completes nearly twice as quick compared to the original test. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/16396#discussion_r1453527575
