From 5073500d39bd2e34331d6303d4fbc7ca892a1715 Mon Sep 17 00:00:00 2001 From: Mohamed Boudra Date: Thu, 10 Sep 2026 10:01:00 +0200 Subject: [PATCH] test(desktop): exit Electron after screenshot verification CI completed every screenshot assertion but remained alive after app.quit. Exit the standalone test explicitly and keep its deadline active until process shutdown. --- packages/desktop/e2e/browser-throttling.electron.cjs | 9 ++++----- 1 file changed, 4 insertions(+), 5 deletions(-) diff --git a/packages/desktop/e2e/browser-throttling.electron.cjs b/packages/desktop/e2e/browser-throttling.electron.cjs index 98d5649a61..3e6fe79a8b 100644 --- a/packages/desktop/e2e/browser-throttling.electron.cjs +++ b/packages/desktop/e2e/browser-throttling.electron.cjs @@ -145,17 +145,16 @@ async function verifyBrowserLifecycle() { } } -const deadline = setTimeout(() => { +setTimeout(() => { console.error("Browser throttling regression timed out"); app.exit(1); }, 30000); app .whenReady() .then(verifyBrowserLifecycle) - .then(() => app.quit()) + .then(() => app.exit(0)) .catch((error) => { console.error(error); app.exit(1); - }) - .finally(() => clearTimeout(deadline)); -app.on("will-quit", () => fs.rmSync(profile, { recursive: true, force: true })); + }); +app.on("quit", () => fs.rmSync(profile, { recursive: true, force: true }));