Changing error message for EMPTY_SPEC_FILES

Author: karanshah-browserstack

bin/helpers/capabilityHelper.js

@@ -98,7 +98,7 @@ const validate = (bsConfig, args) => {
 
     if (!bsConfig.run_settings) reject(Constants.validationMessages.EMPTY_RUN_SETTINGS);
 
-    if (!bsConfig.run_settings.cypress_proj_dir) reject(Constants.validationMessages.EMPTY_SPEC_FILES);
+    if (!bsConfig.run_settings.cypress_proj_dir) reject(Constants.validationMessages.EMPTY_CYPRESS_PROJ_DIR);
 
     // validate parallels specified in browserstack.json if parallels are not specified via arguments
     if (!Utils.isUndefined(args) && Utils.isUndefined(args.parallels) && !Utils.isParallelValid(bsConfig.run_settings.parallels)) reject(Constants.validationMessages.INVALID_PARALLELS_CONFIGURATION);

bin/helpers/constants.js

@@ -23,7 +23,7 @@ const validationMessages = {
   EMPTY_TEST_SUITE: "Test suite is empty",
   EMPTY_BROWSERSTACK_JSON: "Empty browserstack.json",
   EMPTY_RUN_SETTINGS: "Empty run settings",
-  EMPTY_SPEC_FILES: "No spec files specified in run_settings",
+  EMPTY_CYPRESS_PROJ_DIR: "cypress_proj_dir is not set in run_settings. See https://www.browserstack.com/docs/automate/cypress/sample-tutorial to learn more.",
   VALIDATED: "browserstack.json file is validated",
   NOT_VALID: "browerstack.json is not valid",
   NOT_VALID_JSON: "browerstack.json is not a valid json",

bin/helpers/utils.js

@@ -34,8 +34,8 @@ exports.getErrorCodeFromMsg = (errMsg) => {
     case Constants.validationMessages.EMPTY_RUN_SETTINGS:
       errorCode = "bstack_json_invalid_no_run_settings";
       break;
-    case Constants.validationMessages.EMPTY_SPEC_FILES:
-      errorCode = "bstack_json_invalid_values";
+    case Constants.validationMessages.EMPTY_CYPRESS_PROJ_DIR:
+      errorCode = "bstack_json_invalid_no_cypress_proj_dir";
       break;
     case Constants.validationMessages.INVALID_DEFAULT_AUTH_PARAMS:
       errorCode = "bstack_json_default_auth_keys";

test/unit/bin/helpers/capabilityHelper.js

@@ -611,7 +611,7 @@ describe("capabilityHelper.js", () => {
         .catch((error) => {
           chai.assert.equal(
             error,
-            Constants.validationMessages.EMPTY_SPEC_FILES
+            Constants.validationMessages.EMPTY_CYPRESS_PROJ_DIR
           );
         });
     });

test/unit/bin/helpers/utils.js

@@ -25,7 +25,7 @@ describe("utils", () => {
       expect(utils.getErrorCodeFromMsg(constant.validationMessages.INCORRECT_AUTH_PARAMS)).to.eq("bstack_json_invalid_missing_keys");
       expect(utils.getErrorCodeFromMsg(constant.validationMessages.EMPTY_BROWSER_LIST)).to.eq("bstack_json_invalid_no_browsers");
       expect(utils.getErrorCodeFromMsg(constant.validationMessages.EMPTY_RUN_SETTINGS)).to.eq("bstack_json_invalid_no_run_settings");
-      expect(utils.getErrorCodeFromMsg(constant.validationMessages.EMPTY_SPEC_FILES)).to.eq("bstack_json_invalid_values");
+      expect(utils.getErrorCodeFromMsg(constant.validationMessages.EMPTY_CYPRESS_PROJ_DIR)).to.eq("bstack_json_invalid_no_cypress_proj_dir");
       expect(utils.getErrorCodeFromMsg(constant.validationMessages.INVALID_DEFAULT_AUTH_PARAMS)).to.eq("bstack_json_default_auth_keys");
       expect(utils.getErrorCodeFromMsg(constant.validationMessages.INVALID_PARALLELS_CONFIGURATION)).to.eq("invalid_parallels_specified");
       expect(utils.getErrorCodeFromMsg(constant.validationMessages.LOCAL_NOT_SET)).to.eq("cypress_json_base_url_no_local");