qase-javascript-commons 2.0.0-beta.10 → 2.0.0-beta.12

package/changelog.md

@@ -1,3 +1,20 @@
+# qase-javascript-commons@2.0.0-beta.12
+
+## What's new
+
+Fixed an issue when the result has empty step action. Now the reporter will mark such steps as "Unnamed step" and will
+log a warning message.
+
+# qase-javascript-commons@2.0.0-beta.11
+
+## What's new
+
+* The `useV2` option in the reporter's configuration will now enable using the experimental v2 API.
+  Before this fix, v1 API was used despite the configuration.
+
+* Attachments from test steps will now be uploaded to Qase.
+  Before this fix, the reporter uploaded only the attachments made outside of any step scope.
+
 # qase-javascript-commons@2.0.0-beta.10
 
 ## What's new

package/dist/qase.js

@@ -252,7 +252,7 @@ class QaseReporter {
         const { frameworkPackage, frameworkName, reporterName, environment, report = {}, testops = {}, } = options;
         switch (mode) {
             case options_1.ModeEnum.testops: {
-                const { api: { token, headers, ...api } = {}, project, run: { title, description, ...run } = {}, plan = {}, batch = {}, uploadAttachments, } = testops;
+                const { api: { token, headers, ...api } = {}, project, run: { title, description, ...run } = {}, plan = {}, batch = {}, useV2, uploadAttachments, } = testops;
                 if (!token) {
                     throw new Error(`Either "testops.api.token" parameter or "${env_1.EnvApiEnum.token}" environment variable is required in "testops" mode`);
                 }
@@ -277,6 +277,7 @@ class QaseReporter {
                     },
                     plan,
                     batch,
+                    useV2,
                 }, apiClient, typeof environment === 'number' ? environment : undefined);
             }
             case options_1.ModeEnum.report: {

package/dist/reporters/testops-reporter.d.ts

@@ -147,16 +147,19 @@ export declare class TestOpsReporter extends AbstractReporter {
     private getRelation;
     /**
      * @param {TestStepType[]} steps
+     * @param testTitle
      * @returns Promise<ResultStep[]>
      * @private
      */
     private transformSteps;
     /**
      * @param {TestStepType[]} steps
+     * @param testTitle
      * @returns Promise<TestStepResultCreate[]>
      * @private
      */
     private transformStepsV1;
+    private logEmptyStep;
     /**
      * @param {number} runId
      * @returns {Promise<void>}

package/dist/reporters/testops-reporter.js

@@ -146,7 +146,7 @@ class TestOpsReporter extends abstract_reporter_1.AbstractReporter {
      */
     async transformTestResult(result) {
         const attachments = await this.uploadAttachments(result.attachments);
-        const steps = await this.transformSteps(result.steps);
+        const steps = await this.transformSteps(result.steps, result.title);
         const model = {
             title: result.title,
             execution: this.getExecution(result.execution),
@@ -167,7 +167,7 @@ class TestOpsReporter extends abstract_reporter_1.AbstractReporter {
      */
     async transformTestResultV1(result) {
         const attachments = await this.uploadAttachments(result.attachments);
-        const steps = await this.transformStepsV1(result.steps);
+        const steps = await this.transformStepsV1(result.steps, result.title);
         const resultCreate = {
             attachments: attachments,
             comment: result.message,
@@ -231,25 +231,32 @@ class TestOpsReporter extends abstract_reporter_1.AbstractReporter {
     }
     /**
      * @param {TestStepType[]} steps
+     * @param testTitle
      * @returns Promise<ResultStep[]>
      * @private
      */
-    async transformSteps(steps) {
+    async transformSteps(steps, testTitle) {
         const resultsSteps = [];
         for (const step of steps) {
             const attachmentHashes = await this.uploadAttachments(step.attachments);
             const resultStep = {
                 data: {
                     action: '',
-                    attachments: attachmentHashes,
                 },
                 execution: {
                     status: TestOpsReporter.stepStatusMap[step.execution.status],
+                    attachments: attachmentHashes,
                 },
             };
             if (step.step_type === models_1.StepType.TEXT) {
                 if ('action' in step.data && resultStep.data != undefined) {
-                    resultStep.data.action = step.data.action;
+                    if (step.data.action === '') {
+                        this.logEmptyStep(testTitle);
+                        resultStep.data.action = 'Unnamed step';
+                    }
+                    else {
+                        resultStep.data.action = step.data.action;
+                    }
                 }
             }
             if (step.step_type === models_1.StepType.GHERKIN) {
@@ -258,7 +265,7 @@ class TestOpsReporter extends abstract_reporter_1.AbstractReporter {
                 }
             }
             if (step.steps.length > 0) {
-                resultStep.steps = await this.transformSteps(step.steps);
+                resultStep.steps = await this.transformSteps(step.steps, testTitle);
             }
             resultsSteps.push(resultStep);
         }
@@ -266,10 +273,11 @@ class TestOpsReporter extends abstract_reporter_1.AbstractReporter {
     }
     /**
      * @param {TestStepType[]} steps
+     * @param testTitle
      * @returns Promise<TestStepResultCreate[]>
      * @private
      */
-    async transformStepsV1(steps) {
+    async transformStepsV1(steps, testTitle) {
         const resultsSteps = [];
         for (const step of steps) {
             const attachmentHashes = await this.uploadAttachments(step.attachments);
@@ -279,7 +287,13 @@ class TestOpsReporter extends abstract_reporter_1.AbstractReporter {
             };
             if (step.step_type === models_1.StepType.TEXT) {
                 if ('action' in step.data) {
-                    resultStep.action = step.data.action;
+                    if (step.data.action === '') {
+                        this.logEmptyStep(testTitle);
+                        resultStep.action = 'Unnamed step';
+                    }
+                    else {
+                        resultStep.action = step.data.action;
+                    }
                 }
             }
             if (step.step_type === models_1.StepType.GHERKIN) {
@@ -288,12 +302,15 @@ class TestOpsReporter extends abstract_reporter_1.AbstractReporter {
                 }
             }
             if (step.steps.length > 0) {
-                resultStep.steps = await this.transformStepsV1(step.steps);
+                resultStep.steps = await this.transformStepsV1(step.steps, testTitle);
             }
             resultsSteps.push(resultStep);
         }
         return resultsSteps;
     }
+    logEmptyStep(testTitle) {
+        this.logger.log((0, chalk_1.default) `{magenta Test '${testTitle}' has empty action in step. The reporter will mark this step as unnamed step.}`);
+    }
     /**
      * @param {number} runId
      * @returns {Promise<void>}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "qase-javascript-commons",
-  "version": "2.0.0-beta.10",
+  "version": "2.0.0-beta.12",
   "description": "Qase JS Reporters",
   "main": "./dist/index.js",
   "types": "./dist/index.d.ts",