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

package/changelog.md

@@ -1,3 +1,10 @@
+# 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

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,10 +231,11 @@ 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);
@@ -249,7 +250,13 @@ class TestOpsReporter extends abstract_reporter_1.AbstractReporter {
             };
             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.11",
+  "version": "2.0.0-beta.12",
   "description": "Qase JS Reporters",
   "main": "./dist/index.js",
   "types": "./dist/index.d.ts",