From 2220bb3febea4363b30e5d8e54381f84b5302b2f Mon Sep 17 00:00:00 2001 From: Vladimir Sheremet Date: Sun, 2 Jun 2024 21:14:53 +0200 Subject: [PATCH] fix(browser): don't get stuck after the manual page refresh Fixes #5822 --- packages/browser/src/client/orchestrator.ts | 11 ++++------- 1 file changed, 4 insertions(+), 7 deletions(-) diff --git a/packages/browser/src/client/orchestrator.ts b/packages/browser/src/client/orchestrator.ts index f67fbccf17e2..5c9afed05581 100644 --- a/packages/browser/src/client/orchestrator.ts +++ b/packages/browser/src/client/orchestrator.ts @@ -96,12 +96,6 @@ client.ws.addEventListener('open', async () => { debug('test files', testFiles.join(', ')) - // TODO: fail tests suite because no tests found? - if (!testFiles.length) { - await done() - return - } - const runningFiles = new Set(testFiles) channel.addEventListener('message', async (e: MessageEvent): Promise => { @@ -168,7 +162,10 @@ client.ws.addEventListener('open', async () => { } }) - await createTesters(testFiles) + // if page was refreshed, there will be no test files + // createTesters will be called again when tests are running in the UI + if (testFiles.length) + await createTesters(testFiles) }) async function createTesters(testFiles: string[]) {