monocart-reporter 2.9.2 → 2.9.4

package/lib/generate-report.js

@@ -18,38 +18,26 @@ const generateJson = (outputDir, filename, reportData, options) => {
     reportData.jsonPath = Util.relativePath(jsonPath);
 };
 
-const forEachFile = (dir, callback) => {
-    if (!fs.existsSync(dir)) {
+const generateZip = (outputDir, filename, reportData, options) => {
+
+    if (!options.zip) {
         return;
     }
-    const dirs = [];
-    fs.readdirSync(dir, {
-        withFileTypes: true
-    }).forEach((it) => {
 
-        if (it.isFile()) {
-            callback(it.name, dir);
-            return;
+    let zipPath = path.resolve(outputDir, `${filename}.zip`);
+    if (typeof options.zip === 'string') {
+        zipPath = options.zip;
+        if (!zipPath.endsWith('.zip')) {
+            zipPath += '.zip';
         }
-
-        if (it.isDirectory()) {
-            dirs.push(path.resolve(dir, it.name));
+        const zipDir = path.dirname(zipPath);
+        if (!fs.existsSync(zipDir)) {
+            fs.mkdirSync(zipDir, {
+                recursive: true
+            });
         }
-    });
-
-    for (const subDir of dirs) {
-        forEachFile(subDir, callback);
-    }
-
-};
-
-const generateZip = (outputDir, filename, reportData, options) => {
-
-    if (!options.zip) {
-        return;
     }
 
-    const zipPath = path.resolve(outputDir, `${filename}.zip`);
     const reportFiles = [];
     return new Promise((resolve) => {
         const zipFile = new ZipFile();
@@ -62,7 +50,7 @@ const generateZip = (outputDir, filename, reportData, options) => {
             resolve();
         });
 
-        forEachFile(outputDir, (name, dir) => {
+        Util.forEachFile(outputDir, (name, dir) => {
             const absPath = path.resolve(dir, name);
             const relPath = Util.relativePath(absPath, outputDir);
             reportFiles.push(relPath);

package/lib/index.d.ts

@@ -56,8 +56,12 @@ export type MonocartReporterOptions = {
 
     /** output json file for data only */
     json?: boolean;
-    /** output zip file for all report files */
-    zip?: boolean;
+
+    /** output zip file for all report files
+     * {boolean} using default path
+     * {string} zip file path
+     */
+    zip?: boolean | string;
 
     /**
      *  whether to copy attachments to the reporter output dir, defaults to true

package/lib/merge-data.js

@@ -36,8 +36,22 @@ const unzipDataFile = async (item, num, options) => {
     // Do not forget to close the file once you're done
     await zip.close();
 
-    const filename = path.basename(item, '.zip');
-    return path.resolve(unzipDir, `${filename}.json`);
+    let filename = path.basename(item, '.zip');
+    let jsonPath = path.resolve(unzipDir, `${filename}.json`);
+    if (fs.existsSync(jsonPath)) {
+        return jsonPath;
+    }
+
+    Util.forEachFile(unzipDir, (name, dir) => {
+        if (name.endsWith('.html')) {
+            filename = path.basename(name, '.html');
+            return 'break';
+        }
+    }, true);
+
+    jsonPath = path.resolve(unzipDir, `${filename}.json`);
+
+    return jsonPath;
 };
 
 const getReportData = async (item, num, options) => {

package/lib/utils/util.js

@@ -277,6 +277,50 @@ const Util = {
         }
     },
 
+    // eslint-disable-next-line complexity
+    forEachFile: (dir, callback, shallow) => {
+        if (!fs.existsSync(dir)) {
+            return;
+        }
+
+        const isBreak = (res) => {
+            return res === 'break' || res === false;
+        };
+
+        const dirs = [];
+        const list = fs.readdirSync(dir, {
+            withFileTypes: true
+        });
+
+        for (const item of list) {
+
+            if (item.isFile()) {
+                const res = callback(item.name, dir);
+                if (isBreak(res)) {
+                    return res;
+                }
+                continue;
+            }
+
+            if (item.isDirectory()) {
+                dirs.push(path.resolve(dir, item.name));
+            }
+
+        }
+
+        if (shallow) {
+            return;
+        }
+
+        for (const subDir of dirs) {
+            const res = Util.forEachFile(subDir, callback, shallow);
+            if (isBreak(res)) {
+                return res;
+            }
+        }
+
+    },
+
     getTemplate: function(templatePath) {
         if (!Util.templateCache) {
             Util.templateCache = {};

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "monocart-reporter",
-    "version": "2.9.2",
+    "version": "2.9.4",
     "description": "A playwright test reporter. Shows suites/cases/steps with tree style, markdown annotations, custom columns/formatters/data collection visitors, console logs, style tags, send email.",
     "main": "lib/index.js",
     "bin": {
@@ -47,7 +47,7 @@
         "koa": "^2.15.3",
         "koa-static-resolver": "^1.0.6",
         "lz-utils": "^2.1.0",
-        "monocart-coverage-reports": "^2.11.0",
+        "monocart-coverage-reports": "^2.11.1",
         "monocart-locator": "^1.0.2",
         "nodemailer": "^6.9.15"
     },
@@ -60,7 +60,7 @@
         "axios": "^1.7.7",
         "commander": "^12.1.0",
         "dotenv": "^16.4.5",
-        "eslint": "^9.11.1",
+        "eslint": "^9.12.0",
         "eslint-config-plus": "^2.0.2",
         "eslint-plugin-html": "^8.1.2",
         "eslint-plugin-vue": "^9.28.0",