don't print invalid json in case of local not set

Author: SagarGaniga

bin/commands/runs.js

@@ -122,7 +122,7 @@ module.exports = function run(args) {
 
       // display browserstack.json is not valid only if validation of browserstack.json field has failed, otherwise display just the error message
       // If parallels specified in arguments are invalid do not display browserstack.json is invalid message
-      if (!(err === Constants.validationMessages.INVALID_PARALLELS_CONFIGURATION && !utils.isUndefined(args.parallels))) {
+      if (utils.isJSONInvalid(err, args)) {
         logger.error(Constants.validationMessages.NOT_VALID);
       }
 

bin/helpers/utils.js

@@ -380,3 +380,22 @@ exports.versionChangedMessage = (preferredVersion, actualVersion) => {
   message = message.replace("<actualVersion>", actualVersion);
   return message
 }
+
+exports.isJSONInvalid = (err, args) => {
+  let invalid =  true
+
+  if (err === Constants.validationMessages.INVALID_PARALLELS_CONFIGURATION && !this.isUndefined(args.parallels)) {
+    invalid = false
+  }
+
+  if (this.deleteBaseUrlFromError(err) === this.deleteBaseUrlFromError(Constants.validationMessages.LOCAL_NOT_SET)) {
+    invalid = false
+  }
+
+  return invalid
+}
+
+exports.deleteBaseUrlFromError = (err) => {
+  return err.replace(/To test ([\s\S]*)on BrowserStack/g, 'To test on BrowserStack');
+}
+