EPMRPP-84613 Playwright tags are parsed to the RP's attributes and attached to the tests

CHANGELOG.md

@@ -1,3 +1,6 @@
+## [5.1.7] - 2024-01-22
+### Changed
+- The Playwright's tags are parsed to the RP's attributes and attached to the tests.
 
 ## [5.1.6] - 2023-12-19
 ### Fixed

src/__tests__/reporter/attributesReporting.spec.ts

@@ -37,7 +37,7 @@ describe('attributes reporting', () => {
     titlePath: () => ['', suiteName, 'testTitle'],
   };
 
-  test('reporter.testItems should be updated with attributes', () => {
+  test('@smoke reporter.testItems should be updated with attributes', () => {
     reporter.testItems = new Map([['testItemId', { id: 'tempTestItemId', name: 'testTitle' }]]);
     // @ts-ignore
     reporter.addAttributes(attributes, testCase);
@@ -47,7 +47,7 @@ describe('attributes reporting', () => {
     expect(reporter.testItems).toEqual(expectedTestItems);
   });
 
-  test('reporter.suitesInfo should be with attributes', () => {
+  test('@smoke reporter.suitesInfo should be with attributes', () => {
     // @ts-ignore
     reporter.addAttributes(attributes, testCase, suiteName);
     const expectedSuitesInfo = new Map([

src/__tests__/reporter/launchesReporting.spec.ts

@@ -37,12 +37,12 @@ describe('start launch', () => {
 
     beforeAll(() => reporter.onBegin());
 
-    test('client.startLaunch should be called with corresponding params', () => {
+    test('@smoke client.startLaunch should be called with corresponding params', () => {
       expect(reporter.client.startLaunch).toHaveBeenCalledTimes(1);
       expect(reporter.client.startLaunch).toHaveBeenCalledWith(startLaunchObj);
     });
 
-    test('reporter.launchId should be set', () => {
+    test('@smoke reporter.launchId should be set', () => {
       expect(reporter.launchId).toEqual('tempLaunchId');
     });
   });
@@ -64,12 +64,12 @@ describe('start launch', () => {
 
     beforeAll(() => reporter.onBegin());
 
-    test('client.startLaunch should be called with corresponding params', () => {
+    test('@smoke client.startLaunch should be called with corresponding params', () => {
       expect(reporter.client.startLaunch).toHaveBeenCalledTimes(1);
       expect(reporter.client.startLaunch).toHaveBeenCalledWith(startLaunchObj);
     });
 
-    test('reporter.launchId should be set', () => {
+    test('@smoke reporter.launchId should be set', () => {
       expect(reporter.launchId).toEqual('tempLaunchId');
     });
   });
@@ -92,12 +92,12 @@ describe('start launch', () => {
 
     beforeAll(() => reporter.onBegin());
 
-    test('client.startLaunch should be called with corresponding params', () => {
+    test('@smoke client.startLaunch should be called with corresponding params', () => {
       expect(reporter.client.startLaunch).toHaveBeenCalledTimes(1);
       expect(reporter.client.startLaunch).toHaveBeenCalledWith(startLaunchObj);
     });
 
-    test('reporter.launchId should be set', () => {
+    test('@smoke reporter.launchId should be set', () => {
       expect(reporter.launchId).toEqual('tempLaunchId');
     });
   });
@@ -125,12 +125,12 @@ describe('start launch', () => {
       delete process.env.RP_LAUNCH_ID;
     });
 
-    test('client.startLaunch should be called with corresponding params', () => {
+    test('@smoke client.startLaunch should be called with corresponding params', () => {
       expect(reporter.client.startLaunch).toHaveBeenCalledTimes(1);
       expect(reporter.client.startLaunch).toHaveBeenCalledWith(startLaunchObj);
     });
 
-    test('reporter.launchId should be set', () => {
+    test('@smoke reporter.launchId should be set', () => {
       expect(reporter.launchId).toEqual('tempLaunchId');
     });
   });
@@ -144,7 +144,7 @@ describe('finish launch', () => {
 
     beforeAll(() => reporter.onEnd());
 
-    test('launch should be finished', () => {
+    test('@smoke launch should be finished', () => {
       expect(reporter.client.finishLaunch).toHaveBeenCalledTimes(1);
       expect(reporter.client.finishLaunch).toHaveBeenCalledWith('tempLaunchId', {
         endTime: mockedDate,
@@ -164,7 +164,7 @@ describe('finish launch', () => {
 
     beforeAll(() => reporter.onEnd());
 
-    test('launch finish request should not be sent', () => {
+    test('@smoke launch finish request should not be sent', () => {
       expect(reporter.client.finishLaunch).toHaveBeenCalledTimes(0);
       expect(reporter.isLaunchFinishSend).toBe(true);
     });
@@ -186,7 +186,7 @@ describe('finish launch', () => {
       delete process.env.RP_LAUNCH_ID;
     });
 
-    test('launch finish request should not be sent', () => {
+    test('@smoke launch finish request should not be sent', () => {
       expect(reporter.client.finishLaunch).toHaveBeenCalledTimes(0);
       expect(reporter.isLaunchFinishSend).toBe(true);
     });

src/__tests__/reporter/retriesReporting.spec.ts

@@ -38,7 +38,7 @@ describe('retries reporting', () => {
 
   const spyStartTestItem = jest.spyOn(reporter.client, 'startTestItem');
 
-  test('client.startTestItem should be called with retry=true params', () => {
+  test('@smoke client.startTestItem should be called with retry=true params', () => {
     const parentId = 'tempTestItemId';
     const expectedTestObj: StartTestObjType = {
       startTime: reporter.client.helpers.now(),

src/__tests__/reporter/startSuiteTestReporting.spec.ts

@@ -75,7 +75,7 @@ describe('start reporting suite/test', () => {
     jest.clearAllMocks();
   });
 
-  test('client.startTestItem should be called with corresponding params to report suites and test item', () => {
+  test('@smoke client.startTestItem should be called with corresponding params to report suites and test item', () => {
     const expectedSuites = new Map([
       [
         rootSuite,
@@ -149,7 +149,7 @@ describe('start reporting suite/test', () => {
     expect(reporter.testItems).toEqual(expectedTestItems);
   });
 
-  test('client.startTestItem should not be called in case of launch finish request have been send', () => {
+  test('@smoke client.startTestItem should not be called in case of launch finish request have been send', () => {
     reporter.isLaunchFinishSend = true;
     // @ts-ignore
     reporter.onTestBegin(testCase);

src/reporter.ts

@@ -94,6 +94,8 @@ export class RPReporter implements Reporter {
       extendTestDescriptionWithLastError: true,
       ...config,
       launchId: process.env.RP_LAUNCH_ID || config.launchId,
+      // TODO: consider the "grep" and "grep-invert" options
+
     };
     this.suites = new Map();
     this.suitesInfo = new Map();
@@ -375,8 +377,9 @@ export class RPReporter implements Reporter {
         !includePlaywrightProjectNameToCodeReference && playwrightProjectName,
       );
       const { id: parentId } = parentSuiteObj;
+      const name = test.title.replace(/^@\w+\s/, '');
       const startTestItem: StartTestObjType = {
-        name: test.title,
+        name, 
         startTime: this.client.helpers.now(),
         type: TEST_ITEM_TYPES.STEP,
         codeRef,