Feat/playwright improvements

example/playwright/basic.js

@@ -21,3 +21,11 @@ test.describe.parallel('Main suite parallel option', async () => {
     });
   });
 });
+
+test.describe.serial('Main suite serial option', async () => {
+  test('test case #1', async ({ page }) => {
+    await test.step('[Check 1] Open page and confirm title', async () => {
+      await page.goto('https://todomvc.com/examples/vanilla-es6/');
+    });
+  });
+});

src/lib/frameworks/jest.js

@@ -68,9 +68,9 @@ module.exports = (ast, file = '', source = '', opts = {}) => {
         if (['describe', 'it', 'context', 'test'].includes(name)) {
           const line = getLineNumber(path);
           throw new CommentError(
-            'Exclusive tests detected. `.only` call found in ' +
-              `${file}:${line}\n` +
-              'Remove `.only` to restore test checks',
+            'Exclusive tests detected. `.only` call found in '
+              + `${file}:${line}\n`
+              + 'Remove `.only` to restore test checks',
           );
         }
       }
@@ -137,10 +137,10 @@ module.exports = (ast, file = '', source = '', opts = {}) => {
 
         code = noHooks
           ? getCode(source, getLineNumber(path), getEndLineNumber(path), isLineNumber)
-          : beforeEachCode +
-            beforeCode +
-            getCode(source, getLineNumber(path), getEndLineNumber(path), isLineNumber) +
-            afterCode;
+          : beforeEachCode
+            + beforeCode
+            + getCode(source, getLineNumber(path), getEndLineNumber(path), isLineNumber)
+            + afterCode;
 
         const testName = getStringValue(path.parent);
         tests.push({

src/lib/frameworks/playwright.js

@@ -66,13 +66,23 @@ module.exports = (ast, file = '', source = '', opts = {}) => {
         addSuite(path.parentPath.container);
       }
 
+      if (path.isIdentifier({ name: 'serial' })) {
+        if (!path.parentPath && !path.parentPath.container) return;
+        if (!hasStringOrTemplateArgument(path.parentPath.container)) return;
+        addSuite(path.parentPath.container);
+      }
+
       // forbid only
       if (path.isIdentifier({ name: 'only' })) {
         if (!path.parent || !path.parent.object) {
           return;
         }
 
-        const name = path.parent.object.name || path.parent.object.callee.object.name;
+        const name =
+          path.parent?.object?.name ||
+          path.parent?.object?.callee?.object?.name ||
+          path.container?.object?.property?.name;
+
         if (['describe', 'it', 'context', 'test'].includes(name)) {
           const line = getLineNumber(path);
           throw new CommentError(

src/lib/utils.js

@@ -118,8 +118,7 @@ function replaceAtPoint(subject, replaceAt, replaceTo) {
   if (updateLine.includes('|')) {
     lines[replaceAt.line - 1] = updateLine.replace(' |', `${replaceTo} |`);
   } else {
-    lines[replaceAt.line - 1] =
-      updateLine.substring(0, replaceAt.column) + replaceTo + updateLine.substring(replaceAt.column);
+    lines[replaceAt.line - 1] = updateLine.substring(0, replaceAt.column) + replaceTo + updateLine.substring(replaceAt.column);
   }
   return lines.join('\n');
 }

tests/playwright_test.js

@@ -3,6 +3,7 @@ const tsParser = require('@typescript-eslint/typescript-estree');
 const fs = require('fs');
 const { expect } = require('chai');
 const playwrightParser = require('../src/lib/frameworks/playwright');
+const { assert } = require('console');
 
 let source;
 let ast;
@@ -13,11 +14,38 @@ describe('playwright parser', () => {
     ast = jsParser.parse(source, { sourceType: 'unambiguous' });
     const tests = playwrightParser(ast, '', source);
     expect(tests[0]).to.include.key('code');
-    expect(tests.length).to.equal(3);
+    expect(tests.length).to.equal(4);
     expect(tests[0].code).to.include("test('basic");
     expect(tests[0].name).to.equal('basic test');
   });
 
+  it('should forbid describe.only tests', () => {
+    source = `
+test.describe.only('my test', () => {
+  test('my test', async ({ page }) => {
+    await page.goto('https://playwright.dev/');
+  });
+}); 
+    `;
+
+    const program = tsParser.parse(source, {
+      sourceType: 'unambiguous',
+      loc: true,
+      range: true,
+      tokens: true,
+    });
+    ast = {
+      program,
+      type: 'File',
+    };
+    try {
+      playwrightParser(ast, '', source);
+      assert.fail('Expected an error');
+    } catch (err) {
+      expect(err.message).to.include('Exclusive tests detected');
+    }
+  });
+
   it('should parse basic playwright-ts tests', () => {
     source = fs.readFileSync('./example/playwright/basic.ts').toString();
     const program = tsParser.parse(source, {
@@ -357,6 +385,13 @@ describe('playwright parser', () => {
 
       expect(tests[2].suites[0]).to.equal('Main suite parallel option');
     });
+    it('should return suite name if used test.describe.serial mode', () => {
+      source = fs.readFileSync('./example/playwright/basic.js').toString();
+      ast = jsParser.parse(source, { sourceType: 'unambiguous' });
+      const tests = playwrightParser(ast, '', source);
+
+      expect(tests[3].suites[0]).to.equal('Main suite serial option');
+    });
   });
 
   context('test with --line-numbers option', () => {