The --no-wrap logic enahancements.

- Added util for setting the no-wrap on process level as a BOOL val.
- Moved the no-wrap logic to centralizeds place.
- Added default value and the type accepted in the help text for yargs.

Author: suryart

bin/commands/runs.js

@@ -54,6 +54,9 @@ module.exports = function run(args) {
     // run test in headed mode
     utils.setHeaded(bsConfig, args);
 
+    // set the no-wrap
+    utils.setNoWrap(bsConfig, args);
+
     // Validate browserstack.json values and parallels specified via arguments
     return capabilityHelper.validate(bsConfig, args).then(function (cypressJson) {
 
@@ -119,7 +122,7 @@ module.exports = function run(args) {
             utils.sendUsageReport(bsConfig, args, err, Constants.messageTypes.ERROR, 'build_failed');
           });
         }).catch(function (err) {
-          // Zip Upload failed | Local Start failed 
+          // Zip Upload failed | Local Start failed
           logger.error(err);
           if(err === Constants.userMessages.LOCAL_START_FAILED){
             utils.sendUsageReport(bsConfig, args, `${err}\n${Constants.userMessages.LOCAL_START_FAILED}`, Constants.messageTypes.ERROR, 'local_start_failed');

bin/helpers/constants.js

@@ -105,7 +105,7 @@ const cliMessages = {
     LOCAL_MODE: 'Accepted values: ("always-on" | "on-demand") - if you choose to keep the binary "always-on", it will speed up your tests by keeping the Local connection warmed up in the background; otherwise, you can choose to have it spawn and killed for every build',
     LOCAL_IDENTIFIER: "Accepted values: String - assign an identifier to your Local process instance",
     LOCAL_CONFIG_FILE: "Accepted values: String - path to local config-file to your Local process instance. Learn more at https://www.browserstack.com/local-testing/binary-params",
-    NO_WRAP: "Wrap the spec names in --sync mode in case of smaller terminal window size pass --no-wrap"
+    SYNC_NO_WRAP: "Wrap the spec names in --sync mode in case of smaller terminal window size pass --no-wrap"
   },
   COMMON: {
     DISABLE_USAGE_REPORTING: "Disable usage reporting",

bin/helpers/sync/syncSpecsLogs.js

@@ -13,8 +13,12 @@ let specSummary = {
   "specs": [],
   "duration": null
 }
-let noWrap = utils.searchForOption('--no-wrap')
+let noWrap = (process.env.SYNC_NO_WRAP || false);
 let terminalWidth = (process.stdout.columns) * 0.9;
+let lineSeparator = "\n--------------------------------------------------------------------------------";
+if (noWrap) {
+  lineSeparator = "\n" + "-".repeat(terminalWidth);
+}
 
 let  getOptions = (auth, build_id) => {
   return {
@@ -32,14 +36,25 @@ let  getOptions = (auth, build_id) => {
 }
 
 let getTableConfig = () => {
+  let centerWidth = Math.ceil(terminalWidth * 0.01),
+      leftWidth = Math.floor(terminalWidth * 0.75),
+      colWidth = Math.floor(terminalWidth * 0.2);
+
+  // Do not autosize on terminal's width if no-wrap provided
+  if (noWrap) {
+    centerWidth = 1;
+    leftWidth = 100;
+    colWidth = 30;
+  }
+
   return {
     border: getBorderConfig(),
     columns: {
-      1: {alignment: 'center', width: noWrap ? 1 : Math.ceil(terminalWidth * 0.01)},
-      2: {alignment: 'left', width: noWrap ? 100 : Math.floor(terminalWidth * 0.75)}
+      1: {alignment: 'center', width: centerWidth},
+      2: {alignment: 'left', width: leftWidth}
     },
     columnDefault: {
-      width: noWrap ? 30 : Math.floor(terminalWidth * 0.2),
+      width: colWidth,
     },
     columnCount: 3,
   };
@@ -82,7 +97,7 @@ let printSpecsStatus = (bsConfig, buildDetails) => {
         whileProcess(callback)
       },
       function(err, result) { // when loop ends
-        noWrap ? logger.info("\n--------------------------------------------------------------------------------") : logger.info("\n" + "-".repeat(terminalWidth))
+        logger.info(lineSeparator);
         specSummary.duration =  endTime - startTime
         resolve(specSummary)
       }
@@ -140,7 +155,7 @@ let showSpecsStatus = (data) => {
 
 let printInitialLog = () => {
   logger.info(`\n${Constants.syncCLI.LOGS.INIT_LOG}`)
-  noWrap ? logger.info("\n--------------------------------------------------------------------------------") : logger.info("\n" + "-".repeat(terminalWidth))
+  logger.info(lineSeparator);
   n = Constants.syncCLI.INITIAL_DELAY_MULTIPLIER
 }
 

bin/helpers/utils.js

@@ -571,6 +571,14 @@ exports.setHeaded = (bsConfig, args) => {
   }
 };
 
+exports.setNoWrap = (_bsConfig, args) => {
+  if (args.noWrap === true) {
+    process.env.SYNC_NO_WRAP = true;
+  } else {
+    process.env.SYNC_NO_WRAP = false;
+  }
+}
+
 exports.getNumberOfSpecFiles = (bsConfig, args, cypressJson) => {
   let testFolderPath = cypressJson.integrationFolder || Constants.DEFAULT_CYPRESS_SPEC_PATH;
   let globSearchPattern = this.sanitizeSpecsPattern(bsConfig.run_settings.specs) || `${testFolderPath}/**/*.+(${Constants.specFileTypes.join("|")})`;

bin/runner.js

@@ -223,7 +223,9 @@ var argv = yargs
           type: "string"
         },
         'no-wrap': {
-          describe: Constants.cliMessages.RUN.NO_WRAP
+          default: false,
+          describe: Constants.cliMessages.RUN.SYNC_NO_WRAP,
+          type: "boolean"
         }
       })
       .help('help')