From 3f51da147cb93b34f8769dfe3e463cfae397bfd2 Mon Sep 17 00:00:00 2001 From: Tim van der Meij Date: Thu, 18 Apr 2024 15:22:58 +0200 Subject: [PATCH] Remove `waitForTimeout` usage from the `clearInput` helper function We should wait until the input field's value is actually empty instead of waiting for a fixed time (which could lead to intermittent failures). --- test/integration/test_utils.mjs | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/test/integration/test_utils.mjs b/test/integration/test_utils.mjs index 18d8b9540..909b2e31e 100644 --- a/test/integration/test_utils.mjs +++ b/test/integration/test_utils.mjs @@ -115,8 +115,9 @@ async function clearInput(page, selector) { await page.click(selector); await kbSelectAll(page); await page.keyboard.press("Backspace"); - // eslint-disable-next-line no-restricted-syntax - await waitForTimeout(10); + await page.waitForFunction( + `document.querySelector('${selector}').value === ""` + ); } function getSelector(id) {