Merge pull request #1029 from 07souravkunda/fix_serial_run_bu

fix: port already in use bu

Author: karanshah-browserstack

bin/testObservability/reporter/index.js

@@ -270,6 +270,16 @@ class MyReporter {
 
       const port = process.env.REPORTER_API_PORT_NO;
 
+      this.httpServer.on('error', (error) => {
+        if (error.code === 'EADDRINUSE') {
+          debugOnConsole(`Port ${port} is already in use. HTTP server could not start.`);
+          debug(`Port ${port} is already in use. HTTP server could not start.`, true, error);
+        } else {
+          debugOnConsole(`Exception in starting reporter server : ${error}`);
+          debug(`Exception in starting reporter server : ${error}`, true, error);
+        }
+      });
+
       this.httpServer.listen(port, '127.0.0.1', async () => {
         console.log(`Reporter HTTP server listening on port ${port}`);
       });

bin/testhub/utils.js

@@ -5,7 +5,7 @@ const TESTHUB_CONSTANTS = require("./constants");
 const testObservabilityHelper = require("../../bin/testObservability/helper/helper");
 const helper = require("../helpers/helper");
 const accessibilityHelper = require("../accessibility-automation/helper");
-const { detect } = require('detect-port');
+const detectPort = require('detect-port');
 
 
 const isUndefined = (value) => value === undefined || value === null;
@@ -195,17 +195,18 @@ exports.logBuildError = (error, product = "") => {
   }
 };
 
-exports.findAvailablePort = async (preferredPort, maxAttempts = 10) => {  
+exports.findAvailablePort = async (preferredPort, maxAttempts = 10) => {
+  const findPort = detectPort.detect || detectPort;
   let port = preferredPort;
   for (let attempts = 0; attempts < maxAttempts; attempts++) {
     try {
-      const availablePort = await detect(port);
+      const availablePort = await findPort(port);
 
       if (availablePort === port) {
           return port;
       } else {
         // Double-check suggested port
-        const verify = await detect(availablePort);
+        const verify = await findPort(availablePort);
         if (verify === availablePort) {
           return availablePort;
         }