artes 1.2.25 → 1.2.27

package/cucumber.config.js

@@ -60,11 +60,13 @@ module.exports = {
     timeout: process.env.TIMEOUT
       ? Number(process.env.TIMEOUT) * 1000
       : artesConfig.timeout * 1000 || 30 * 1000, // Default timeout in seconds
-    paths: process.env.FEATURES
-      ? [path.join(moduleConfig.projectPath, process.env.FEATURES)]
-      : artesConfig.features
-        ? path.join(moduleConfig.projectPath, artesConfig.features)
-        : [moduleConfig.featuresPath], // Paths to feature files
+    paths: process.env.RERUN
+      ? [`${path.join("../../", process.env.RERUN)}`]
+      : process.env.FEATURES
+        ? [path.join(moduleConfig.projectPath, process.env.FEATURES)]
+        : artesConfig.features
+          ? [path.join(moduleConfig.projectPath, artesConfig.features)]
+          : [moduleConfig.featuresPath], // Paths to feature files
     require: [
       process.env.STEP_DEFINITIONS
         ? [path.join(moduleConfig.projectPath, process.env.STEP_DEFINITIONS)]

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "artes",
-  "version": "1.2.25",
+  "version": "1.2.27",
   "description": "The simplest way to automate UI and API tests using Cucumber-style steps.",
   "main": "index.js",
   "scripts": {
@@ -23,9 +23,9 @@
     "@faker-js/faker": "9.8.0",
     "@types/node": "22.9.0",
     "ajv": "8.17.1",
-    "allure-commandline": "2.30.0",
-    "allure-cucumberjs": "3.0.5",
-    "allure-js-commons": "3.0.5",
+    "allure-commandline": "2.36.0",
+    "allure-cucumberjs": "3.4.5",
+    "allure-js-commons": "3.4.5",
     "archiver": "^7.0.1",
     "dayjs": "1.11.13",
     "deasync": "^0.1.31",

package/src/helper/controller/elementController.js

@@ -12,7 +12,7 @@ function addElements(newElements) {
 //   return locatorCount ==0 ? false : true;
 // }
 
-function selectorSeperator(element) {
+function selectorSeparator(element) {
   if (typeof element !== "string") return element;
 
   const selector = element?.split("=");
@@ -41,7 +41,7 @@ function selectorSeperator(element) {
 function getSelector(element) {
   const selector =
     elements?.[element]?.selector || elements?.[element] || element;
-  return resolveVariable(selectorSeperator(selector));
+  return resolveVariable(selectorSeparator(selector));
 }
 
 function getElement(element) {

package/src/helper/controller/pomCollector.js

@@ -1,25 +1,53 @@
 const { addElements } = require("./elementController");
 const cucumberConfig = require("../../../cucumber.config");
 const fs = require("fs");
+const path = require("path");
+
+const duplicateWarnings = [];
+const keyRegistry = {};
 
 function pomCollector() {
-  if (fs.existsSync(cucumberConfig.default.pomPath)) {
-    fs.readdir(`${cucumberConfig.default.pomPath}`, (err, files) => {
-      files.forEach((file) => {
-        fs.readFile(
-          `${cucumberConfig.default.pomPath}/${file}`,
-          "utf-8",
-          (err, content) => {
-            try {
-              addElements(JSON.parse(content));
-            } catch (error) {
-              console.log(`Error parsing POM file ${file}:`, error.message);
-            }
-          },
+  const pomPath = cucumberConfig.default.pomPath;
+
+  if (!fs.existsSync(pomPath)) return;
+
+  fs.readdirSync(pomPath).forEach((file) => {
+    const filePath = path.join(pomPath, file);
+
+    let parsed;
+    try {
+      parsed = JSON.parse(fs.readFileSync(filePath, "utf-8"));
+    } catch (error) {
+      console.log(`Error parsing POM file ${file}: ${error.message}`);
+      return;
+    }
+
+    Object.keys(parsed).forEach((key) => {
+      if (keyRegistry[key]) {
+        duplicateWarnings.push(
+          `${key} in ${file} has the same key with ${key} in ${keyRegistry[key]}`
         );
-      });
+      } else {
+        keyRegistry[key] = file;
+      }
     });
-  }
+
+    addElements(parsed);
+  });
+}
+
+function logPomWarnings() {
+  if (duplicateWarnings.length === 0) return;
+
+  console.warn(
+    "\n\x1b[33m[WARNING] POM DUPLICATE KEY WARNINGS:  This may break your tests or cause flaky behavior."
+  ); 
+  
+  duplicateWarnings.forEach((warning) => {
+    console.warn(`- ${warning}`);
+  });
+
+  console.log("\x1b[0m")
 }
 
-module.exports = { pomCollector };
+module.exports = { pomCollector, logPomWarnings };

package/src/helper/executers/testRunner.js

@@ -5,15 +5,10 @@ const path = require("path");
 function runTests() {
   try {
     console.log("🧪 Running tests...");
-    process.env.FORCE_COLOR = "1";
-    process.env.FORCE_STDIO_TTY = "1";
 
     spawnSync(
       "cucumber-js",
-      [
-        "--config=cucumber.config.js",
-        `${process.env.RERUN ? path.join("../../", process.env.RERUN) : ""}`,
-      ],
+      [ "--config=cucumber.config.js"],
       {
         cwd: moduleConfig.modulePath,
         stdio: "inherit",

package/src/hooks/hooks.js

@@ -11,7 +11,7 @@ const {
 const { spawnSync } = require("child_process");
 const { invokeBrowser } = require("../helper/contextManager/browserManager");
 const { invokeRequest } = require("../helper/contextManager/requestManager");
-const { pomCollector } = require("../helper/controller/pomCollector");
+const { pomCollector, logPomWarnings } = require("../helper/controller/pomCollector");
 const cucumberConfig = require("../../cucumber.config");
 const { context } = require("./context");
 const fs = require("fs");
@@ -156,9 +156,10 @@ After(async function ({ pickle, result }) {
       path: screenshotPath,
       type: "png",
     });
+
     await this.attach(img, {
       mediaType: "image/png",
-      fileName: `${pickle.name.replaceAll(" ", "_")}.png`,
+      fileName: "Screenshot",
     });
   }
 
@@ -184,7 +185,7 @@ After(async function ({ pickle, result }) {
 
       await this.attach(trace, {
         mediaType: "application/zip",
-        fileName: `${pickle.name.replace(/\s+/g, "_")}_trace.zip`,
+        fileName: "Trace",
       });
 
       if (!cucumberConfig.default.trace) {
@@ -227,7 +228,7 @@ After(async function ({ pickle, result }) {
         const webmBuffer = fs.readFileSync(videoPath);
         await this.attach(webmBuffer, {
           mediaType: "video/webm",
-          fileName: `${pickle.name.replaceAll(" ", "_")}.webm`,
+          fileName: "Screenrecord",
         });
       }
     }
@@ -239,6 +240,8 @@ AfterAll(async () => {
     await projectHooks.AfterAll();
   }
 
+  logPomWarnings();
+
   if (!fs.existsSync(statusDir)) return;
 
   const files = fs.readdirSync(statusDir);
@@ -256,9 +259,8 @@ AfterAll(async () => {
     });
   }
 
-  if (cucumberConfig.default.testPercentage !== undefined) {
-    const meetsThreshold =
-      successPercentage >= cucumberConfig.default.testPercentage;
+  if (cucumberConfig.default.testPercentage>0) {
+    const meetsThreshold = successPercentage >= cucumberConfig.default.testPercentage;
 
     if (meetsThreshold) {
       console.log(