mocha-qase-reporter 1.0.0-beta.3 → 1.0.0-beta.5

package/README.md

@@ -89,7 +89,7 @@ QASE_MODE=testops mocha --parallel
 After the tests are finished, you can complete the run:
 
 ```bash
-qli testops run complete --project DEMO --token token --run $(echo $QASE_TESTOPS_RUN_ID)
+qli testops run complete --project DEMO --token token --id $(echo $QASE_TESTOPS_RUN_ID)
 ```
 
 ## Configuration

package/changelog.md

@@ -1,3 +1,17 @@
+# qase-mocha@1.0.0-beta.5
+
+## What's new
+
+- Added a `qase` function to allow specifying QaseID for tests.
+- Marked the old syntax for QaseID as deprecated.
+- Implemented functionality to capture console logs and include them as attachments in tests.
+
+# qase-mocha@1.0.0-beta.4
+
+## What's new
+
+Fixed the issue with async tests not being reported correctly.
+
 # qase-mocha@1.0.0-beta.3
 
 ## What's new

package/dist/interceptor.d.ts

@@ -0,0 +1,12 @@
+/// <reference types="node" />
+/// <reference types="node" />
+import { Writable } from 'stream';
+export interface TestOutput {
+    stdout: string;
+    stderr: string;
+}
+export declare class StreamInterceptor extends Writable {
+    private readonly onWrite;
+    constructor(onWriteCallback: (data: string) => void);
+    _write(chunk: any, _encoding: BufferEncoding, callback: (error?: Error | null) => void): void;
+}

package/dist/interceptor.js

@@ -0,0 +1,16 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.StreamInterceptor = void 0;
+const stream_1 = require("stream");
+class StreamInterceptor extends stream_1.Writable {
+    constructor(onWriteCallback) {
+        super();
+        this.onWrite = onWriteCallback;
+    }
+    _write(chunk, _encoding, callback) {
+        // eslint-disable-next-line @typescript-eslint/no-unsafe-member-access,@typescript-eslint/no-unsafe-argument,@typescript-eslint/no-unsafe-call
+        this.onWrite(chunk.toString());
+        callback();
+    }
+}
+exports.StreamInterceptor = StreamInterceptor;

package/dist/mocha.d.ts

@@ -0,0 +1,12 @@
+/**
+ * Set IDs for the test case
+ *
+ * @param caseId
+ * @param name
+ * @example
+ * it(qase(1, 'test'), function() => {
+ *  // test code
+ * });
+ * @returns {string}
+ */
+export declare const qase: (caseId: number | string | number[] | string[], name: string) => string;

package/dist/mocha.js

@@ -0,0 +1,32 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.qase = void 0;
+/**
+ * Set IDs for the test case
+ *
+ * @param caseId
+ * @param name
+ * @example
+ * it(qase(1, 'test'), function() => {
+ *  // test code
+ * });
+ * @returns {string}
+ */
+const qase = (caseId, name) => {
+    const caseIds = Array.isArray(caseId) ? caseId : [caseId];
+    const ids = [];
+    for (const id of caseIds) {
+        if (typeof id === 'number') {
+            ids.push(id);
+            continue;
+        }
+        const parsedId = parseInt(id);
+        if (!isNaN(parsedId)) {
+            ids.push(parsedId);
+            continue;
+        }
+        console.log(`qase: qase ID ${id} should be a number`);
+    }
+    return `${name} (Qase ID: ${caseIds.join(',')})`;
+};
+exports.qase = qase;

package/dist/reporter.d.ts

@@ -3,6 +3,9 @@ import { MochaOptions, reporters, Runner } from 'mocha';
 import { ConfigLoader, TestStatusEnum } from 'qase-javascript-commons';
 type MochaState = 'failed' | 'passed' | 'pending';
 export declare class MochaQaseReporter extends reporters.Base {
+    private originalStdoutWrite;
+    private originalStderrWrite;
+    private testOutputs;
     /**
      * @type {Record<CypressState, TestStatusEnum>}
      */
@@ -23,6 +26,7 @@ export declare class MochaQaseReporter extends reporters.Base {
     private applyListeners;
     private onStartRun;
     private onEndRun;
+    private addMethodsToContext;
     private addMethods;
     private onStartTest;
     private onEndTest;
@@ -63,5 +67,21 @@ export declare class MochaQaseReporter extends reporters.Base {
     }) => void;
     comment: (message: string) => void;
     step: (title: string, func: () => void) => void;
+    /**
+     * @param {string} title
+     * @returns {string}
+     * @private
+     */
+    private removeQaseIdsFromTitle;
+    /**
+     * @type {RegExp}
+     */
+    static qaseIdRegExp: RegExp;
+    /**
+     * @param {string} title
+     * @returns {number[]}
+     * @private
+     */
+    private static getCaseId;
 }
 export {};

package/dist/reporter.js

@@ -10,6 +10,7 @@ const qase_javascript_commons_1 = require("qase-javascript-commons");
 const deasync_promise_1 = __importDefault(require("deasync-promise"));
 const node_path_1 = require("node:path");
 const uuid_1 = require("uuid");
+const interceptor_1 = require("./interceptor");
 const Events = mocha_1.Runner.constants;
 class currentTest {
     constructor() {
@@ -33,9 +34,10 @@ class MochaQaseReporter extends mocha_1.reporters.Base {
         this.applyListeners = () => {
             this.runner.on(Events.EVENT_RUN_BEGIN, () => this.onStartRun());
             this.runner.on(Events.EVENT_RUN_END, () => this.onEndRun());
-            this.runner.on(Events.EVENT_TEST_BEGIN, (test) => this.addMethods(test.ctx));
-            this.runner.on(Events.EVENT_HOOK_BEGIN, (hook) => this.addMethods(hook.ctx));
+            this.runner.on(Events.EVENT_TEST_BEGIN, (test) => this.addMethods(test));
+            this.runner.on(Events.EVENT_HOOK_BEGIN, (hook) => this.addMethodsToContext(hook.ctx));
             this.runner.on(Events.EVENT_TEST_BEGIN, () => this.onStartTest());
+            // eslint-disable-next-line @typescript-eslint/no-misused-promises
             this.runner.on(Events.EVENT_TEST_END, (test) => this.onEndTest(test));
         };
         this.qaseId = (id) => {
@@ -122,6 +124,9 @@ class MochaQaseReporter extends mocha_1.reporters.Base {
         else {
             this.applyListeners();
         }
+        this.originalStdoutWrite = process.stdout.write.bind(process.stdout);
+        this.originalStderrWrite = process.stderr.write.bind(process.stderr);
+        this.testOutputs = new Map();
     }
     onStartRun() {
         this.reporter.startTestRun();
@@ -129,7 +134,7 @@ class MochaQaseReporter extends mocha_1.reporters.Base {
     onEndRun() {
         (0, deasync_promise_1.default)(this.reporter.publish());
     }
-    addMethods(ctx) {
+    addMethodsToContext(ctx) {
         if (!ctx)
             return;
         ctx.qaseId = this.qaseId;
@@ -143,14 +148,50 @@ class MochaQaseReporter extends mocha_1.reporters.Base {
         ctx.comment = this.comment;
         ctx.step = this.step;
     }
+    addMethods(test) {
+        const stdoutInterceptor = new interceptor_1.StreamInterceptor((data) => {
+            const output = this.testOutputs.get(test.title) ?? { stdout: '', stderr: '' };
+            output.stdout += data;
+            this.testOutputs.set(test.title, output);
+        });
+        const stderrInterceptor = new interceptor_1.StreamInterceptor((data) => {
+            const output = this.testOutputs.get(test.title) ?? { stdout: '', stderr: '' };
+            output.stderr += data;
+            this.testOutputs.set(test.title, output);
+        });
+        // eslint-disable-next-line @typescript-eslint/ban-ts-comment
+        // @ts-ignore
+        process.stdout.write = stdoutInterceptor.write.bind(stdoutInterceptor);
+        // eslint-disable-next-line @typescript-eslint/ban-ts-comment
+        // @ts-ignore
+        process.stderr.write = stderrInterceptor.write.bind(stderrInterceptor);
+        this.testOutputs.set(test.title, { stdout: '', stderr: '' });
+        this.addMethodsToContext(test.ctx);
+    }
     onStartTest() {
         this.currentType = 'test';
     }
     onEndTest(test) {
+        process.stdout.write = this.originalStdoutWrite;
+        process.stderr.write = this.originalStderrWrite;
+        if (this.reporter.isCaptureLogs()) {
+            const output = this.testOutputs.get(test.title);
+            if (output?.stdout) {
+                this.attach({ name: 'stdout.txt', content: output.stdout, contentType: 'text/plain' });
+            }
+            if (output?.stderr) {
+                this.attach({ name: 'stderr.txt', content: output.stderr, contentType: 'text/plain' });
+            }
+        }
         if (this.metadata.ignore) {
+            this.metadata.clear();
+            this.currentTest = new currentTest();
             return;
         }
         const ids = this.getQaseId();
+        if (ids.length === 0) {
+            ids.push(...MochaQaseReporter.getCaseId(test.title));
+        }
         const suites = this.getSuites(test);
         let relations = {};
         if (suites.length > 0) {
@@ -195,7 +236,7 @@ class MochaQaseReporter extends mocha_1.reporters.Base {
                 thread: null,
             },
             testops_id: ids.length > 0 ? ids : null,
-            title: this.metadata.title && this.metadata.title != '' ? this.metadata.title : test.title,
+            title: this.metadata.title && this.metadata.title != '' ? this.metadata.title : this.removeQaseIdsFromTitle(test.title),
         };
         (0, deasync_promise_1.default)(this.reporter.addTestResult(result));
         this.metadata.clear();
@@ -263,6 +304,27 @@ class MochaQaseReporter extends mocha_1.reporters.Base {
         }
         return suites;
     }
+    /**
+     * @param {string} title
+     * @returns {string}
+     * @private
+     */
+    removeQaseIdsFromTitle(title) {
+        const matches = title.match(MochaQaseReporter.qaseIdRegExp);
+        if (matches) {
+            return title.replace(matches[0], '').trimEnd();
+        }
+        return title;
+    }
+    /**
+     * @param {string} title
+     * @returns {number[]}
+     * @private
+     */
+    static getCaseId(title) {
+        const [, ids] = title.match(MochaQaseReporter.qaseIdRegExp) ?? [];
+        return ids ? ids.split(',').map((id) => Number(id)) : [];
+    }
 }
 exports.MochaQaseReporter = MochaQaseReporter;
 /**
@@ -273,3 +335,7 @@ MochaQaseReporter.statusMap = {
     passed: qase_javascript_commons_1.TestStatusEnum.passed,
     pending: qase_javascript_commons_1.TestStatusEnum.skipped,
 };
+/**
+ * @type {RegExp}
+ */
+MochaQaseReporter.qaseIdRegExp = /\(Qase ID: ([\d,]+)\)/;

package/dist/types.d.ts

@@ -13,6 +13,10 @@ export interface Hook extends Mocha.Hook {
     ctx?: Context;
 }
 export interface Methods {
+    /**
+     * Set IDs for the test case
+     * Use `qase()` instead. This method is deprecated and kept for reverse compatibility.
+     */
     qaseId(id: number | number[]): void;
     title(title: string): void;
     parameters(values: Record<string, string>): void;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "mocha-qase-reporter",
-  "version": "1.0.0-beta.3",
+  "version": "1.0.0-beta.5",
   "description": "Mocha Cypress Reporter",
   "homepage": "https://github.com/qase-tms/qase-javascript",
   "sideEffects": false,
@@ -9,7 +9,8 @@
   "exports": {
     ".": "./dist/index.js",
     "./reporter": "./dist/reporter.js",
-    "./package.json": "./package.json"
+    "./package.json": "./package.json",
+    "./mocha": "./dist/mocha.js"
   },
   "typesVersions": {
     "*": {