Made syntax more clearer

Author: pawanpanjwani

bin/commands/runs.js

@@ -16,7 +16,7 @@ module.exports = function run(args) {
     utils.setUsageReportingFlag(bsConfig, args.disableUsageReporting);
 
     // Validate browserstack.json values
-    return capabilityHelper.validate(bsConfig).then(function (validated) {
+    return capabilityHelper.validate(bsConfig, args).then(function (validated) {
       logger.info(validated);
 
       // accept the number of parallels

bin/helpers/capabilityHelper.js

@@ -96,18 +96,11 @@ const validate = (bsConfig, args) => {
     if (!bsConfig.run_settings.cypress_proj_dir) reject(Constants.validationMessages.EMPTY_SPEC_FILES);
 
     // validate parallels specified in browserstack.json if parallels are not specified via arguments
-    if (Utils.isUndefined(args.parallels) && !Utils.isParallelValid(bsConfig.run_settings.parallels)) {
-      reject(Constants.validationMessages.INVALID_PARALLELS_CONFIGURATION);
-    }
-    // if parallels specified via arguments validate both parallels specifed in browserstack.json and parallels specified in arguments
+    if (Utils.isUndefined(args.parallels) && !Utils.isParallelValid(bsConfig.run_settings.parallels)) reject(Constants.validationMessages.INVALID_PARALLELS_CONFIGURATION);
+  
+    // if parallels specified via arguments validate only arguments
     if (!Utils.isUndefined(args.parallels) && !Utils.isParallelValid(args.parallels)) reject(Constants.validationMessages.INVALID_PARALLELS_CONFIGURATION);
 
-
-    if (!args.parallels) {
-      if (Number.isNaN(bsConfig.run_settings.parallels) || (parseInt(bsConfig.run_settings.parallels, 10) < 0 && parseInt(bsConfig.run_settings.parallels, 10) != -1))
-        reject(Constants.validationMessages.INVALID_PARALLELS_CONFIGURATION)
-    };
-
     resolve(Constants.validationMessages.VALIDATED);
   });
 }

bin/helpers/utils.js

@@ -71,7 +71,7 @@ exports.setUsageReportingFlag = (bsConfig, disableUsageReporting) => {
 }
 
 exports.setParallels = (bsConfig, args) => {
-  if ((args.parallels !== undefined && !isNaN(args.parallels)) && bsConfig && (bsConfig['run_settings']['parallels'] === undefined || bsConfig['run_settings']['parallels'])) {
+  if (!this.isUndefined(args.parallels)) {
     bsConfig['run_settings']['parallels'] = args.parallels;
   }
 }