Add warn in constants

Author: prajwaldhawarikar-bs

bin/helpers/capabilityHelper.js

@@ -287,7 +287,7 @@ const validate = (bsConfig, args) => {
         logger.warn(Constants.validationMessages.NODE_VERSION_PARSING_ERROR);
 
     if(!Utils.isUndefined(cypressConfigFile.port)) {
-      logger.warn(`The requested port number ${cypressConfigFile.port} is ignored. The default BrowserStack port will be used for this execution`);
+      logger.warn(Constants.userMessages.CYPRESS_PORT_WARNING.replace("<x>", cypressConfigFile.port));
     }
     resolve(cypressConfigFile);
   });

bin/helpers/constants.js

@@ -113,7 +113,9 @@ const userMessages = {
   SPEC_LIMIT_SUCCESS_MESSAGE:
     "Spec timeout specified as <x> minutes. If any of your specs exceed the specified time limit, it would be forcibly killed by BrowserStack",
   NO_CONNECTION_WHILE_UPDATING_UPLOAD_PROGRESS_BAR:
-    "Unable to determine zip upload progress due to undefined/null connection request"
+    "Unable to determine zip upload progress due to undefined/null connection request",
+  CYPRESS_PORT_WARNING:
+    "The requested port number <x> is ignored. The default BrowserStack port will be used for this execution"
 };
 
 const validationMessages = {

test/unit/bin/helpers/capabilityHelper.js

@@ -1063,7 +1063,7 @@ describe("capabilityHelper.js", () => {
         await capabilityHelper
           .validate(bsConfig, { parallels: 2 })
 
-        sinon.assert.calledWith(loggerWarningSpy, `The requested port number 23455 is ignored. The default BrowserStack port will be used for this execution`);
+        sinon.assert.calledWith(loggerWarningSpy, Constants.userMessages.CYPRESS_PORT_WARNING.replace('<x>', 23455));
         fs.existsSync.restore();
         fs.readFileSync.restore();
       });