🐛 fix for addinf cypress dependency

Author: Archish27

bin/helpers/checkUploaded.js

@@ -86,20 +86,7 @@ const checkUploadedMd5 = (bsConfig, args, instrumentBlocks) => {
       zipUrlPresent: false,
       packageUrlPresent: false,
     };
-    const runSettings = bsConfig.run_settings;
-    if (runSettings.npm_dependencies !== undefined && 
-      Object.keys(runSettings.npm_dependencies).length !== 0 &&
-      typeof runSettings.npm_dependencies === 'object') {
-      if (!("cypress" in runSettings.npm_dependencies)) {
-        logger.warn("Missing cypress not found in npm_dependencies");
-        if ("cypress_version" in runSettings && !runSettings.cypress_version.toString().match(Constants.LATEST_VERSION_SYNTAX_REGEX)) {
-          runSettings.npm_dependencies.cypress = runSettings.cypress_version;
-        } else {
-          runSettings.npm_dependencies.cypress = "latest";
-        }
-        logger.warn(`Adding cypress version ${runSettings.npm_dependencies.cypress} in npm_dependencies`);
-      }
-    }
+    utils.setCypressNpmDependency(bsConfig);
     if (args["force-upload"]) {
       logger.debug("force-upload set to true. Uploading tests and npm packages.");
       return resolve(obj);

bin/helpers/utils.js

@@ -328,6 +328,27 @@ exports.setCypressTestSuiteType = (bsConfig) => {
   logger.debug(`Setting cypress test suite type as ${bsConfig.run_settings.cypressTestSuiteType}`);
 }
 
+exports.setCypressNpmDependency = (bsConfig) => {
+  const runSettings = bsConfig.run_settings;
+  if (runSettings.npm_dependencies !== undefined && 
+    Object.keys(runSettings.npm_dependencies).length !== 0 &&
+    typeof runSettings.npm_dependencies === 'object') {
+    if (!("cypress" in runSettings.npm_dependencies)) {
+      logger.warn("Missing cypress not found in npm_dependencies");        
+      if("cypress_version" in runSettings){
+        if(runSettings.cypress_version.toString().match(Constants.LATEST_VERSION_SYNTAX_REGEX)){
+          runSettings.npm_dependencies.cypress = `^${runSettings.cypress_version.toString().split(".")[0]}`
+        } else {
+          runSettings.npm_dependencies.cypress = runSettings.cypress_version;
+        }
+      } else if (runSettings.cypressTestSuiteType === Constants.CYPRESS_V10_AND_ABOVE_TYPE)  {
+        runSettings.npm_dependencies.cypress = "latest";
+      }
+      logger.warn(`Adding cypress version ${runSettings.npm_dependencies.cypress} in npm_dependencies`);
+    }
+  }
+}
+
 exports.verifyGeolocationOption = () => {
   let glOptionsSet = (this.searchForOption('-gl') || this.searchForOption('--gl'));
   let geoHyphenLocationOptionsSet = (this.searchForOption('-geo-location') || this.searchForOption('--geo-location'));

test/unit/bin/helpers/utils.js

@@ -23,6 +23,7 @@ const utils = require('../../../../bin/helpers/utils'),
   syncLogger = require('../../../../bin/helpers/logger').syncCliLogger,
   Contants = require('../../../../bin/helpers/constants');
 const browserstack = require('browserstack-local');
+const { CYPRESS_V10_AND_ABOVE_TYPE, CYPRESS_V9_AND_OLDER_TYPE } = require('../../../../bin/helpers/constants');
 chai.use(chaiAsPromised);
 logger.transports['console.info'].silent = true;
 
@@ -3522,4 +3523,82 @@ describe('utils', () => {
       expect(utils.getMajorVersion('4.1')).to.be.eql('4');
     });
   });
+
+  describe('#setCypressNpmDependency', () => {
+    
+    it('should set cypress as latest for cypress 10 test suite if cypress_version missing', () => {
+      let bsConfig = {
+        run_settings: {
+          cypressConfigFilePath: 'cypress.json',
+          npm_dependencies: {
+            "dummy": "verison"
+          },
+          cypressTestSuiteType: CYPRESS_V10_AND_ABOVE_TYPE
+        },        
+      };
+      utils.setCypressNpmDependency(bsConfig);
+      chai.assert.equal(bsConfig.run_settings.npm_dependencies.cypress, "latest");
+    });
+
+    it('should set cypress as ^10 if cypress version added', () => {
+      let bsConfig = {
+        run_settings: {
+          cypress_version: "10.latest",
+          cypressConfigFilePath: 'cypress.json',
+          npm_dependencies: {
+            "dummy": "verison"
+          },
+          cypressTestSuiteType: CYPRESS_V10_AND_ABOVE_TYPE
+        },        
+      };
+      utils.setCypressNpmDependency(bsConfig);
+      chai.assert.equal(bsConfig.run_settings.npm_dependencies.cypress, "^10");
+    });
+
+    it('should set cypress as ^10 if cypress version added', () => {
+      let bsConfig = {
+        run_settings: {
+          cypress_version: "10.latest",
+          cypressConfigFilePath: 'cypress.json',
+          npm_dependencies: {
+            "dummy": "verison"
+          },
+          cypressTestSuiteType: CYPRESS_V10_AND_ABOVE_TYPE
+        },        
+      };
+      utils.setCypressNpmDependency(bsConfig);
+      chai.assert.equal(bsConfig.run_settings.npm_dependencies.cypress, "^10");
+    });
+
+    it('should set cypress as 10.0.0 if cypress version added', () => {
+      let bsConfig = {
+        run_settings: {
+          cypress_version: "10.0.0",
+          cypressConfigFilePath: 'cypress.json',
+          npm_dependencies: {
+            "dummy": "verison"
+          },
+          cypressTestSuiteType: CYPRESS_V10_AND_ABOVE_TYPE
+        },        
+      };
+      utils.setCypressNpmDependency(bsConfig);
+      chai.assert.equal(bsConfig.run_settings.npm_dependencies.cypress, "10.0.0");
+    });
+
+    it('should not set cypress for < 9 cypress version if cypress_version missing', () => {
+      let bsConfig = {
+        run_settings: {
+          cypressConfigFilePath: 'cypress.json',
+          npm_dependencies: {
+            "dummy": "verison"
+          },
+          cypressTestSuiteType: CYPRESS_V9_AND_OLDER_TYPE
+        },        
+      };
+      utils.setCypressNpmDependency(bsConfig);
+      chai.assert.equal(bsConfig.run_settings.npm_dependencies.cypress, undefined);
+    });
+  });
+
+    
 });