added validations

Author: roshan04

bin/helpers/capabilityHelper.js

@@ -242,6 +242,23 @@ const validate = (bsConfig, args) => {
       addCypressZipStartLocation(bsConfig.run_settings);
     }
 
+    // check if Interactive Capabilities Caps passed is correct or not
+    if(!Utils.isUndefined(bsConfig.run_settings.interactive_debugging) && !Utils.isUndefined(bsConfig.run_settings.interactiveDebugging)) {
+      if(Utils.isConflictingBooleanValues(bsConfig.run_settings.interactive_debugging, bsConfig.run_settings.interactiveDebugging)) {
+        reject(Constants.userMessages.CYPRESS_INTERACTIVE_SESSION_CONFLICT_VALUES);
+      } else if(Utils.isNonBooleanValue(bsConfig.run_settings.interactive_debugging) && Utils.isNonBooleanValue(bsConfig.run_settings.interactiveDebugging)) {
+        logger.warn('You have passed an invalid value to the interactive_debugging capability. Proceeding with the default value (True).');
+      }
+    } else if(!Utils.isUndefined(bsConfig.run_settings.interactive_debugging)) {
+      if(Utils.isNonBooleanValue(bsConfig.run_settings.interactive_debugging)) {
+        logger.warn('You have passed an invalid value to the interactive_debugging capability. Proceeding with the default value (True).');
+      }
+    } else if(!Utils.isUndefined(bsConfig.run_settings.interactiveDebugging)) {
+      if(Utils.isNonBooleanValue(bsConfig.run_settings.interactiveDebugging)) {
+        logger.warn('You have passed an invalid value to the interactive_debugging capability. Proceeding with the default value (True).');
+      }
+    }
+
     // check if two config files are present at the same location
     let cypressFileDirectory = path.dirname(path.resolve(bsConfig.run_settings.cypressConfigFilePath));
     let listOfFiles = fs.readdirSync(cypressFileDirectory);

bin/helpers/constants.js

@@ -117,7 +117,9 @@ const userMessages = {
   NO_CONNECTION_WHILE_UPDATING_UPLOAD_PROGRESS_BAR:
     "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"
+    "The requested port number <x> is ignored. The default BrowserStack port will be used for this execution",
+  CYPRESS_INTERACTIVE_SESSION_CONFLICT_VALUES:
+    "Conflicting values (True & False) were found for the interactive_debugging capability. Please resolve this issue to proceed further."
 };
 
 const validationMessages = {

bin/helpers/utils.js

@@ -1000,16 +1000,22 @@ exports.setHeaded = (bsConfig, args) => {
   logger.debug(`headless mode set to ${bsConfig.run_settings.headless}`);
 };
 
+exports.isConflictingBooleanValues = (value1, value2) => {
+  return (value1.toString() == "true" && value2.toString() == "false") || (value1.toString() == "false" && value2.toString() == "true")
+};
+
+exports.isNonBooleanValue = (value) => {
+  return value.toString() != "true" && value.toString() != "false";
+};
+
 exports.setInteractiveCapability = (bsConfig) => {
-  let interactiveDegugging = true;
-  if(bsConfig.interactive_debugging != undefined) {
-    interactiveDegugging = bsConfig.interactive_debugging;
-  }
-  if(bsConfig.interactiveDegugging != undefined) {
-    interactiveDegugging = bsConfig.interactiveDegugging;
-  }
-  console.log(`roshan1: the interactiveDegugging ${interactiveDegugging} ::`);
-  bsConfig.interactiveDegugging = interactiveDegugging;
+  let interactiveDebuggingTemp = "true";
+  let interactive_debugging = bsConfig.run_settings.interactive_debugging;
+  let interactiveDebugging = bsConfig.run_settings.interactiveDebugging;
+  if(!isNonBooleanValue(interactive_debugging)) interactiveDebuggingTemp = interactive_debugging;
+  else if(!isNonBooleanValue(interactiveDebugging)) interactiveDebuggingTemp = interactiveDebugging;
+  logger.debug(`Setting interactiveDebugging flag to ${interactiveDebuggingTemp}`);
+  bsConfig.run_settings.interactiveDebugging = interactiveDebuggingTemp;
 }
 
 exports.setNoWrap = (_bsConfig, args) => {