From c2e9a6264c9cb1f4073309fd87ecfa7cc70a1068 Mon Sep 17 00:00:00 2001 From: Tim van der Meij Date: Thu, 18 Apr 2024 15:31:01 +0200 Subject: [PATCH] Remove `waitForTimeout` usage from the `dragAndDropAnnotation` helper function The timeout was introduced in commit 402e3fe with equal timeouts around the helper function call. In commit 55e5af2 the timeouts around the helper function call have been removed, and it looks like the helper function itself was not updated purely due to an oversight. The operations here should not require any timeouts because the promises only resolve once the action is completed, which also explains why removing the timeouts surrounding the helper function calls went without any problems. It should therefore be safe to remove this timeout too. --- test/integration/test_utils.mjs | 2 -- 1 file changed, 2 deletions(-) diff --git a/test/integration/test_utils.mjs b/test/integration/test_utils.mjs index 909b2e31e..71983fa3d 100644 --- a/test/integration/test_utils.mjs +++ b/test/integration/test_utils.mjs @@ -355,8 +355,6 @@ async function serializeBitmapDimensions(page) { async function dragAndDropAnnotation(page, startX, startY, tX, tY) { await page.mouse.move(startX, startY); await page.mouse.down(); - // eslint-disable-next-line no-restricted-syntax - await waitForTimeout(10); await page.mouse.move(startX + tX, startY + tY); await page.mouse.up(); await page.waitForSelector("#viewer:not(.noUserSelect)");