On Tue, 16 Jan 2024 14:42:07 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. > >> 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. Thank you, the code has been updated. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/16396#discussion_r1453747130
