monocart-reporter 2.9.4 → 2.9.6

package/README.md

@@ -1066,7 +1066,11 @@ npx monocart merge <glob-patterns>
 npx monocart merge path-to/shard*/index.json -o merged-reports/index.html
 
 # -c --config
-npx monocart merge path-to/shard*/index.json -c mr.config.js
+npx monocart merge path-to/shard*/my-report.zip -c mr.config.js
+
+# NOTE: The asterisk(*) is a special character which is interpreted by some operating systems
+# For example: Mac and Linux, please put it in quotes, but NOT for Windows
+npx monocart merge 'path-to/shard*/*.zip'
 ```
 The default config files (In order of priority)
 - mr.config.js

package/lib/cli.js

@@ -1,4 +1,4 @@
-#!/usr/bin/env node
+#!/usr/bin/env node
 
 const fs = require('fs');
 const path = require('path');
@@ -272,6 +272,12 @@ const resolveConfigOptions = async (configPath) => {
     return configOptions;
 };
 
+const exit = (code) => {
+    if (code) {
+        process.exit(code);
+    }
+};
+
 const mergeReports = async (str, cliOptions) => {
 
     await initNodeOptions(cliOptions);
@@ -285,6 +291,7 @@ const mergeReports = async (str, cliOptions) => {
     } else {
         if (customConfig) {
             Util.logError(`ERROR: not found config file: ${customConfig}`);
+            exit(1);
             return;
         }
     }
@@ -297,15 +304,21 @@ const mergeReports = async (str, cliOptions) => {
 
     Util.initLoggingLevel(options.logging);
 
+    Util.logInfo(`glob patterns: ${EC.cyan(str)}`);
+
     const files = await glob(str);
     if (!Util.isList(files)) {
         Util.logError(`ERROR: no files found with glob: ${str}`);
+        exit(1);
         return;
     }
-    // console.log(files, options);
 
-    await merge(files, options);
+    Util.logInfo(`glob files: ${files.join(', ')}`);
 
+    const reportData = await merge(files, options);
+    if (!reportData) {
+        exit(1);
+    }
 };
 
 program

package/lib/generate-report.js

@@ -18,34 +18,80 @@ const generateJson = (outputDir, filename, reportData, options) => {
     reportData.jsonPath = Util.relativePath(jsonPath);
 };
 
-const generateZip = (outputDir, filename, reportData, options) => {
-
-    if (!options.zip) {
-        return;
+const initZipOptions = (outputDir, filename, zipOptions) => {
+    let clean = false;
+    let outputFile;
+    if (typeof zipOptions === 'string') {
+        outputFile = zipOptions;
+    } else if (typeof zipOptions === 'object') {
+        if (zipOptions.outputFile) {
+            outputFile = zipOptions.outputFile;
+        }
+        if (zipOptions.clean) {
+            clean = true;
+        }
     }
 
-    let zipPath = path.resolve(outputDir, `${filename}.zip`);
-    if (typeof options.zip === 'string') {
-        zipPath = options.zip;
-        if (!zipPath.endsWith('.zip')) {
-            zipPath += '.zip';
+    if (outputFile) {
+        if (!outputFile.endsWith('.zip')) {
+            outputFile += '.zip';
         }
-        const zipDir = path.dirname(zipPath);
+        const zipDir = path.dirname(outputFile);
         if (!fs.existsSync(zipDir)) {
             fs.mkdirSync(zipDir, {
                 recursive: true
             });
         }
+
+    } else {
+        outputFile = path.resolve(outputDir, `${filename}.zip`);
+    }
+
+
+    return {
+        outputFile,
+        clean
+    };
+};
+
+const generateZip = (outputDir, filename, reportData, options) => {
+
+    const zipOptions = options.zip;
+    if (!zipOptions) {
+        return;
     }
 
+    const { outputFile, clean } = initZipOptions(outputDir, filename, zipOptions);
+
     const reportFiles = [];
     return new Promise((resolve) => {
         const zipFile = new ZipFile();
-        zipFile.outputStream.pipe(fs.createWriteStream(zipPath)).on('close', function() {
+        zipFile.outputStream.pipe(fs.createWriteStream(outputFile)).on('close', function() {
 
-            // remove files after zip
             reportData.reportFiles = reportFiles;
-            reportData.zipPath = Util.relativePath(zipPath);
+
+            // whether to clean the files after zipped
+            if (clean) {
+                // console.log('clean', reportFiles);
+                const dirSet = new Set();
+                reportFiles.forEach((f) => {
+                    const fp = path.resolve(outputDir, f);
+                    dirSet.add(path.dirname(fp));
+                    Util.rmSync(fp);
+                });
+
+                // clean empty dirs
+                const dirList = Array.from(dirSet).reverse();
+                dirList.forEach((dir) => {
+                    const files = fs.readdirSync(dir);
+                    if (files.length === 0) {
+                        Util.rmSync(dir);
+                    }
+                });
+
+            }
+
+            reportData.zipPath = Util.relativePath(outputFile);
 
             resolve();
         });
@@ -308,6 +354,7 @@ const generateReport = async (reportData, options, rawData) => {
 
     Util.logInfo(`view report: ${EC.cyan(`npx monocart show-report ${htmlPath}`)}`);
 
+    return reportData;
 };
 
 module.exports = generateReport;

package/lib/index.d.ts

@@ -61,7 +61,12 @@ export type MonocartReporterOptions = {
      * {boolean} using default path
      * {string} zip file path
      */
-    zip?: boolean | string;
+    zip?: boolean | string | {
+        /** the zip file path */
+        outputFile?: string;
+        /** whether to clean the files after zipped */
+        clean?: boolean;
+    };
 
     /**
      *  whether to copy attachments to the reporter output dir, defaults to true

package/lib/merge-data.js

@@ -36,20 +36,30 @@ const unzipDataFile = async (item, num, options) => {
     // Do not forget to close the file once you're done
     await zip.close();
 
-    let filename = path.basename(item, '.zip');
+    const filename = path.basename(item, '.zip');
     let jsonPath = path.resolve(unzipDir, `${filename}.json`);
     if (fs.existsSync(jsonPath)) {
         return jsonPath;
     }
 
+    // filename for both html and json
+    const htmlMap = {};
+    const jsonMap = {};
     Util.forEachFile(unzipDir, (name, dir) => {
         if (name.endsWith('.html')) {
-            filename = path.basename(name, '.html');
-            return 'break';
+            htmlMap[path.basename(name, '.html')] = true;
+        } else if (name.endsWith('.json')) {
+            jsonMap[path.basename(name, '.json')] = true;
         }
     }, true);
 
-    jsonPath = path.resolve(unzipDir, `${filename}.json`);
+    const keys = Object.keys(htmlMap);
+    for (const fn of keys) {
+        if (jsonMap[fn]) {
+            jsonPath = path.resolve(unzipDir, `${fn}.json`);
+            break;
+        }
+    }
 
     return jsonPath;
 };
@@ -74,7 +84,7 @@ const getReportData = async (item, num, options) => {
         // for finding attachments
         const jsonDir = Util.relativePath(path.dirname(item));
 
-        Util.logInfo(`report data loaded: ${item}`);
+        Util.logInfo(`report data loaded: ${Util.relativePath(item)}`);
         return {
             jsonDir,
             data

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "monocart-reporter",
-    "version": "2.9.4",
+    "version": "2.9.6",
     "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": {
@@ -53,7 +53,7 @@
     },
     "devDependencies": {
         "@babel/code-frame": "^7.25.7",
-        "@playwright/test": "^1.47.2",
+        "@playwright/test": "^1.48.0",
         "ansi-to-html": "^0.7.2",
         "async-tick": "^1.0.0",
         "autolinker": "^4.0.0",