Adding unit test cases for isCypressProjDirValid

Author: karanshah81295

bin/helpers/utils.js

@@ -140,8 +140,8 @@ exports.isCypressProjDirValid = (cypressDir, cypressProjDir) => {
   // Getting absolute path
   cypressDir = path.resolve(cypressDir);
   cypressProjDir = path.resolve(cypressProjDir);
-
   if(cypressProjDir === cypressDir) return true;
-  let parentTokens = cypressDir.split('/').filter(i => i.length)
-  return parentTokens.every((t, i) => cypressProjDir.split('/')[i] === t)
+  let parentTokens = cypressDir.split('/').filter(i => i.length);
+  let childTokens = cypressProjDir.split('/').filter(i => i.length);
+  return parentTokens.every((t, i) => childTokens[i] === t);
 }

test/unit/bin/helpers/utils.js

@@ -236,4 +236,18 @@ describe("utils", () => {
       sinon.assert.calledOnce(sendUsageReportStub);
     });
   });
+
+  describe("isCypressProjDirValid", () => {
+    it("should return true when cypressDir and cypressProjDir is same", () =>{
+      expect(utils.isCypressProjDirValid("/absolute/path","/absolute/path")).to.be.true;
+    });
+
+    it("should return true when cypressProjDir is child directory of cypressDir", () =>{
+      expect(utils.isCypressProjDirValid("/absolute/path","/absolute/path/childpath")).to.be.true;
+    });
+
+    it("should return false when cypressProjDir is not child directory of cypressDir", () =>{
+      expect(utils.isCypressProjDirValid("/absolute/path","/absolute")).to.be.false;
+    });
+  });
 });