EPMRPP-84449 || Added 'skippedIssue'

__tests__/report-portal-client.spec.js

@@ -34,7 +34,7 @@
 
       client.logDebug('message');
 
       expect(console.log).toHaveBeenCalledWith('message', '');
     });
 
     it('should call console.log with messages if debug is true and dataMsg provided', () => {
@@ -48,7 +48,7 @@
 
       client.logDebug('message', { key: 1, value: 2 });
 
       expect(console.log).toHaveBeenCalledWith('message', { key: 1, value: 2 });
     });
 
     it('should not call console.log if debug is false', () => {
@@ -62,7 +62,7 @@
 
       client.logDebug('message');
 
       expect(console.log).not.toHaveBeenCalled();
     });
   });
 
@@ -876,6 +876,91 @@
     });
   });
 
+  it('should automatically add NOT_ISSUE when status is SKIPPED and skippedIssue is false', function (done) {
+    const mockClient = new RPClient({
+      apiKey: 'test',
+      endpoint: 'https://reportportal-stub-url',
+      launch: 'test launch',
+      project: 'test project',
+      skippedIssue: false,
+    }, { name: 'test', version: '1.0.0' });
+
+    const spyFinishTestItemPromiseStart = jest.spyOn(mockClient, 'finishTestItemPromiseStart').mockImplementation(() => {});
+
+    mockClient.map = {
+      testItemId: {
+        children: [],
+        finishSend: false,
+        promiseFinish: Promise.resolve(),
+        resolveFinish: () => {},
+      },
+    };
+
+    const finishTestItemRQ = {
+      status: 'skipped',
+    };
+
+    mockClient.finishTestItem('testItemId', finishTestItemRQ);
+
+    setTimeout(() => {
+      expect(spyFinishTestItemPromiseStart).toHaveBeenCalledWith(
+        expect.any(Object),
+        'testItemId',
+        expect.objectContaining({
+          status: 'skipped',
+          issue: { issueType: 'NOT_ISSUE' },
+        })
+      );
+      done();
+    }, 50);
+  });
+
+  it('should not add NOT_ISSUE when status is SKIPPED and skippedIssue is true', function (done) {
+    const mockClient = new RPClient({
+      apiKey: 'test',
+      endpoint: 'https://reportportal-stub-url',
+      launch: 'test launch',
+      project: 'test project',
+      skippedIssue: true,
+    }, { name: 'test', version: '1.0.0' });
+
+    const spyFinishTestItemPromiseStart = jest.spyOn(mockClient, 'finishTestItemPromiseStart').mockImplementation(() => {});
+
+    mockClient.map = {
+      testItemId: {
+        children: [],
+        finishSend: false,
+        promiseFinish: Promise.resolve(),
+        resolveFinish: () => {},
+      },
+    };
+
+    const finishTestItemRQ = {
+      status: 'skipped',
+    };
+
+    mockClient.finishTestItem('testItemId', finishTestItemRQ);
+
+    setTimeout(() => {
+      expect(spyFinishTestItemPromiseStart).toHaveBeenCalledWith(
+        expect.any(Object),
+        'testItemId',
+        expect.objectContaining({
+          status: 'skipped',
+        })
+      );
+      expect(spyFinishTestItemPromiseStart).not.toHaveBeenCalledWith(
+        expect.any(Object),
+        'testItemId',
+        expect.objectContaining({
+          issue: expect.anything(),
+        })
+      );
+      done();
+    }, 100);
+
+  });
+
   describe('saveLog', () => {
     it('should return object with tempId and promise', () => {
       const client = new RPClient({ apiKey: 'any', endpoint: 'https://rp.api', project: 'prj' });

index.d.ts

@@ -155,6 +155,7 @@ declare module '@reportportal/client-javascript' {
     launchUuidPrintOutput?: string;
     restClientConfig?: RestClientConfig;
     token?: string;
+    skippedIssue?: boolean;
     /**
      * OAuth 2.0 configuration object. When provided, OAuth authentication will be used instead of API key.
      */

lib/commons/config.js

@@ -24,7 +24,7 @@
     if (!calculatedApiKey) {
       throw new ReportPortalRequiredOptionError('apiKey');
     } else {
       console.warn(`Option 'token' is deprecated. Use 'apiKey' instead.`);
     }
   }
 
@@ -115,6 +115,7 @@
       description: options.description,
       launchUuidPrint: options.launchUuidPrint,
       launchUuidPrintOutput,
+      skippedIssue: options.skippedIssue,
     };
   } catch (error) {
     // don't throw the error up to not break the entire process

lib/report-portal-client.js

@@ -610,6 +610,10 @@ class RPClient {
       ...(itemObj.children.length ? {} : { status: RP_STATUSES.PASSED }),
       ...finishTestItemRQ,
     };
+
+    if (finishTestItemData.status === RP_STATUSES.SKIPPED && this.config.skippedIssue === false) {
+      finishTestItemData.issue = { issueType: 'NOT_ISSUE' };
+    }
 
     itemObj.finishSend = true;
     this.logDebug(`Finish all children for test item with tempId ${itemTempId}`);