fix: define commonPath based on stepBase.

Author: Song-Nan17

lib/getStepDefinitionsPaths.js

@@ -20,7 +20,7 @@ const getStepDefinitionsPaths = filePath => {
     if (config.nonGlobalStepBaseDir) {
       const stepBase = `${appRoot}/${config.nonGlobalStepBaseDir}`;
       nonGlobalPath = nonGlobalPath.replace(stepDefinitionPath(), stepBase);
-      commonPath = `${nonGlobalPath}/../${config.commonPath || "common/"}`;
+      commonPath = `${stepBase}/${config.commonPath || "common/"}`;
     }
 
     const nonGlobalPattern = `${nonGlobalPath}/**/*.+(js|ts)`;

lib/getStepDefinitionsPaths.test.js

@@ -83,7 +83,7 @@ describe("getStepDefinitionsPaths", () => {
       const expectedNonGlobalDefinitionPattern =
         "cwd/nonGlobalStepBaseDir/test/**/*.+(js|ts)";
       const expectedCommonPath =
-        "cwd/nonGlobalStepBaseDir/test/../common/**/*.+(js|ts)";
+        "cwd/nonGlobalStepBaseDir/common/**/*.+(js|ts)";
 
       expect(actual).to.include(expectedNonGlobalDefinitionPattern);
       expect(actual).to.include(expectedCommonPath);
@@ -97,7 +97,7 @@ describe("getStepDefinitionsPaths", () => {
       const actual = getStepDefinitionsPaths(path);
 
       const expectedCommonPath =
-        "cwd/nonGlobalStepBaseDir/test/../commonPath/**/*.+(js|ts)";
+        "cwd/nonGlobalStepBaseDir/commonPath/**/*.+(js|ts)";
 
       expect(actual).to.include(expectedCommonPath);
     });