Improved threading.

Author: iamcharankumar

src/main/java/io/swaglabs/portal/qa/browsermanager/BrowserManager.java

@@ -10,7 +10,7 @@
 public class BrowserManager implements IBrowserManager<Page> {
 
     @Override
-    public Page getBrowserPage(Playwright playwright) {
+    public synchronized Page getBrowserPage(Playwright playwright) {
         boolean isHeadless = "headless".equals(WebPortalConstants.RUN_MODE);
         BrowserContext browserContext = new BrowserFactory().createBrowser().createSession(playwright, isHeadless);
         Objects.requireNonNull(browserContext, "Playwright Browser Context is null!");

src/test/java/io/swaglabs/portal/qa/commons/WebBaseTest.java

@@ -15,7 +15,6 @@
 import org.testng.annotations.BeforeSuite;
 
 import java.lang.reflect.Method;
-import java.util.Objects;
 
 @Slf4j
 public abstract class WebBaseTest {
@@ -38,34 +37,47 @@ public void init(Method method) {
         log.info("Browser has been set.");
         WebTestListeners.setPage(page.get());
         if (isChromiumBrowser()) {
-            CdpUtils.initializeCdpSession(page.get());
-            CdpUtils.enableCdpSession();
-            CdpUtils.sendCommand(CdpCommands.NETWORK_ENABLE.getDescription());
-            CdpUtils.logErrorResponses();
-            CdpUtils.sendCommand(CdpCommands.CONSOLE_ENABLE.getDescription());
-            CdpUtils.logConsoleErrors();
-            CdpUtils.sendCommand(CdpCommands.RUNTIME_ENABLE.getDescription());
-            CdpUtils.logUncaughtJavascriptErrors();
-            CdpUtils.logUncaughtConsoleErrors();
-            CdpUtils.sendCommand(CdpCommands.PAGE_ENABLE.getDescription());
-            CdpUtils.logPageLoadCompletion();
-            CdpUtils.logPageNavigatedWithinDocument();
+            initializeCdpSession(page.get());
         }
+        trackPerformanceMetrics(method);
+    }
+
+    private void initializeCdpSession(Page page) {
+        CdpUtils.initializeCdpSession(page);
+        CdpUtils.enableCdpSession();
+        CdpUtils.sendCommand(CdpCommands.NETWORK_ENABLE.getDescription());
+        CdpUtils.logErrorResponses();
+        CdpUtils.sendCommand(CdpCommands.CONSOLE_ENABLE.getDescription());
+        CdpUtils.logConsoleErrors();
+        CdpUtils.sendCommand(CdpCommands.RUNTIME_ENABLE.getDescription());
+        CdpUtils.logUncaughtJavascriptErrors();
+        CdpUtils.logUncaughtConsoleErrors();
+        CdpUtils.sendCommand(CdpCommands.PAGE_ENABLE.getDescription());
+        CdpUtils.logPageLoadCompletion();
+        CdpUtils.logPageNavigatedWithinDocument();
+    }
+
+    private void trackPerformanceMetrics(Method method) {
         PerformanceUtils.evaluatePageLoadTime(page.get(), method.getName());
         PerformanceUtils.evaluateDomContentLoadTime(page.get(), method.getName());
     }
 
     @AfterMethod(alwaysRun = true)
     public void destroy() {
-        if (isChromiumBrowser()) {
-            CdpUtils.destroyCdpSession();
+        try {
+            if (isChromiumBrowser()) {
+                CdpUtils.destroyCdpSession();
+            }
+            browserManager.destroyBrowserPage(page.get());
+        } catch (Exception e) {
+            log.error("Cleanup failed: ", e);
+        } finally {
+            page.remove();
+            if (PLAYWRIGHT.get() != null) {
+                PLAYWRIGHT.get().close();
+                PLAYWRIGHT.remove();
+            }
         }
-        browserManager.destroyBrowserPage(page.get());
-        page.remove();
-        Objects.requireNonNull(PLAYWRIGHT.get(), "Playwright is null!");
-        PLAYWRIGHT.get().close();
-        PLAYWRIGHT.remove();
-        log.info("Browser has been destroyed.");
     }
 
     private boolean isChromiumBrowser() {