From 397b8de3273e4c3d38409d544993723eeb7731fc Mon Sep 17 00:00:00 2001
From: Noam Gaash <noam.gaash@applitools.com>
Date: Wed, 24 Apr 2024 16:19:34 +0300
Subject: [PATCH] fix: gracefullyClose unused workers

This patch fixes issue #30504 by gracefully terminating the worker before exiting the process
---
 packages/playwright-test/src/common/process.ts | 13 +++++++------
 1 file changed, 7 insertions(+), 6 deletions(-)

diff --git a/packages/playwright-test/src/common/process.ts b/packages/playwright-test/src/common/process.ts
index 8b0cd4222bd3b..4f38147a43f30 100644
--- a/packages/playwright-test/src/common/process.ts
+++ b/packages/playwright-test/src/common/process.ts
@@ -90,12 +90,13 @@ async function gracefullyCloseAndExit() {
   if (closed)
     return;
   closed = true;
-  // Force exit after 30 seconds.
-  setTimeout(() => process.exit(0), 30000);
-  // Meanwhile, try to gracefully shutdown.
-  await processRunner.gracefullyClose().catch(() => {});
-  await stopProfiling(processName).catch(() => {});
-  process.exit(0);
+  try {
+    // try to gracefully shutdown.
+    await processRunner.gracefullyClose().catch(() => {});
+    await stopProfiling(processName).catch(() => {});
+  } finally {
+    process.exit(0);
+  }
 }
 
 function sendMessageToParent(message: { method: string, params?: any }) {