@zohodesk/testinglibrary 0.1.7 → 0.1.8-exp-bdd

package/build/bdd-framework/gen/poms.js

@@ -0,0 +1,46 @@
+"use strict";
+
+Object.defineProperty(exports, "__esModule", {
+  value: true
+});
+exports.POMS = void 0;
+exports.buildFixtureTag = buildFixtureTag;
+var _createDecorators = require("../stepDefinitions/createDecorators");
+/**
+ * Handle POMs graph for decorator steps.
+ */
+
+const FIXTURE_TAG_PREFIX = '@fixture:';
+class POMS {
+  usedPoms = new Map();
+  add(pomNode) {
+    if (pomNode) this.usedPoms.set(pomNode, null);
+  }
+  addByFixtureName(fixtureName) {
+    const pomNode = (0, _createDecorators.getPomNodeByFixtureName)(fixtureName);
+    if (pomNode) this.add(pomNode);
+  }
+  addByTag(tag) {
+    const fixtureName = extractFixtureName(tag);
+    if (fixtureName) this.addByFixtureName(fixtureName);
+  }
+  resolveFixtureNames(pomNode) {
+    const resolvedFixtureNames = this.usedPoms.get(pomNode);
+    if (resolvedFixtureNames) return resolvedFixtureNames;
+    const fixtureNames = [...pomNode.children].map(child => this.resolveFixtureNames(child)).flat();
+    if (this.usedPoms.has(pomNode)) {
+      // if nothing returned from children, use own fixtureName,
+      // otherwise use what returned from child
+      if (!fixtureNames.length) fixtureNames.push(pomNode.fixtureName);
+      this.usedPoms.set(pomNode, fixtureNames);
+    }
+    return fixtureNames;
+  }
+}
+exports.POMS = POMS;
+function extractFixtureName(tag) {
+  return tag.startsWith(FIXTURE_TAG_PREFIX) ? tag.replace(FIXTURE_TAG_PREFIX, '') : '';
+}
+function buildFixtureTag(fixtureName) {
+  return `${FIXTURE_TAG_PREFIX}${fixtureName}`;
+}

package/build/bdd-framework/gen/testFile.js

@@ -10,18 +10,15 @@ var _path = _interopRequireDefault(require("path"));
 var _formatter = require("./formatter");
 var _i18n = require("./i18n");
 var _loadSteps = require("../cucumber/loadSteps");
+var _createBdd = require("../stepDefinitions/createBdd");
 var _index = require("@cucumber/cucumber/lib/formatter/helpers/index");
 var _utils = require("../utils");
 var _testPoms = require("./testPoms");
 var _testNode = require("./testNode");
 var _stepConfig = require("../stepDefinitions/stepConfig");
 var _exit = require("../utils/exit");
-var _fixtures = require("./fixtures");
-var _scenario = require("../hooks/scenario");
-var _worker = require("../hooks/worker");
-var _lang = require("../config/lang");
-/**
- * Generate test code.
+/**
+ * Generate test code.
  */
 
 class TestFile {
@@ -29,7 +26,6 @@ class TestFile {
   lines = [];
   i18nKeywordsMap;
   formatter;
-  hasCucumberStyle = false;
   testNodes = [];
   hasCustomTest = false;
   undefinedSteps = [];
@@ -41,9 +37,7 @@ class TestFile {
     const {
       uri
     } = this.options.doc;
-    if (!uri) {
-      throw new Error(`Document without uri`);
-    }
+    if (!uri) throw new Error(`Document without uri`);
     return uri;
   }
   get content() {
@@ -51,10 +45,7 @@ class TestFile {
   }
   get language() {
     var _this$options$doc$fea;
-    return ((_this$options$doc$fea = this.options.doc.feature) === null || _this$options$doc$fea === void 0 ? void 0 : _this$options$doc$fea.language) || _lang.LANG_EN;
-  }
-  get isEnglish() {
-    return (0, _lang.isEnglish)(this.language);
+    return ((_this$options$doc$fea = this.options.doc.feature) === null || _this$options$doc$fea === void 0 ? void 0 : _this$options$doc$fea.language) || 'en';
   }
   get config() {
     return this.options.config;
@@ -64,18 +55,14 @@ class TestFile {
   }
   build() {
     this.loadI18nKeywords();
-    this.lines = [...this.getFileHeader(),
-    // prettier-ignore
-    ...this.getRootSuite(), ...this.getFileFixtures()];
+    this.lines = [...this.getFileHeader(), ...this.getRootSuite(), ...this.getFileFixtures()];
     return this;
   }
   save() {
     const dir = _path.default.dirname(this.outputPath);
-    if (!_fs.default.existsSync(dir)) {
-      _fs.default.mkdirSync(dir, {
-        recursive: true
-      });
-    }
+    if (!_fs.default.existsSync(dir)) _fs.default.mkdirSync(dir, {
+      recursive: true
+    });
     _fs.default.writeFileSync(this.outputPath, this.content);
   }
   getFileHeader() {
@@ -83,7 +70,7 @@ class TestFile {
     return this.formatter.fileHeader(this.sourceFile, importTestFrom);
   }
   loadI18nKeywords() {
-    if (!this.isEnglish) {
+    if (this.language !== 'en') {
       this.i18nKeywordsMap = (0, _i18n.getKeywordsMap)(this.language);
     }
   }
@@ -91,9 +78,7 @@ class TestFile {
     const {
       importTestFrom
     } = this.config;
-    if (!importTestFrom) {
-      return;
-    }
+    if (!importTestFrom) return;
     const {
       file,
       varName
@@ -105,46 +90,37 @@ class TestFile {
     };
   }
   getFileFixtures() {
-    return this.formatter.useFixtures([...this.formatter.testFixture(), ...(!this.isEnglish ? this.formatter.langFixture(this.language) : []), ...((0, _scenario.hasScenarioHooks)() || this.hasCucumberStyle ? this.formatter.bddWorldFixtures() : []), ...this.formatter.scenarioHookFixtures((0, _scenario.getScenarioHooksFixtures)()), ...this.formatter.workerHookFixtures((0, _worker.getWorkerHooksFixtures)()), ...this.formatter.tagsFixture(this.testNodes)]);
+    return this.formatter.useFixtures([...this.formatter.testFixture(), ...this.formatter.tagsFixture(this.testNodes)]);
   }
   getRootSuite() {
     const {
       feature
     } = this.options.doc;
-    if (!feature) {
-      throw new Error(`Document without feature.`);
-    }
+    if (!feature) throw new Error(`Document without feature.`);
     return this.getSuite(feature);
   }
-  /**
-     * Generate test.describe suite for root Feature or Rule
+  /**
+     * Generate test.describe suite for root Feature or Rule
      */
   getSuite(feature, parent) {
     const node = new _testNode.TestNode(feature, parent);
-    if (node.isSkipped()) {
-      return this.formatter.suite(node, []);
-    }
     const lines = [];
+    // const { backgrounds, rules, scenarios } =
+    // bgFixtures, bgTags - used as fixture hints for decorator steps
     feature.children.forEach(child => lines.push(...this.getSuiteChild(child, node)));
     return this.formatter.suite(node, lines);
   }
   getSuiteChild(child, parent) {
-    if ('rule' in child && child.rule) {
-      return this.getSuite(child.rule, parent);
-    }
-    if (child.background) {
-      return this.getBeforeEach(child.background, parent);
-    }
-    if (child.scenario) {
-      return this.getScenarioLines(child.scenario, parent);
-    }
+    if ('rule' in child && child.rule) return this.getSuite(child.rule, parent);
+    if (child.background) return this.getBeforeEach(child.background, parent);
+    if (child.scenario) return this.getScenarioLines(child.scenario, parent);
     throw new Error(`Empty child: ${JSON.stringify(child)}`);
   }
   getScenarioLines(scenario, parent) {
     return this.isOutline(scenario) ? this.getOutlineSuite(scenario, parent) : this.getTest(scenario, parent);
   }
-  /**
-     * Generate test.beforeEach for Background
+  /**
+     * Generate test.beforeEach for Background
      */
   getBeforeEach(bg, parent) {
     const node = new _testNode.TestNode({
@@ -157,18 +133,15 @@ class TestFile {
     } = this.getSteps(bg, node.tags);
     return this.formatter.beforeEach(fixtures, lines);
   }
-  /**
-     * Generate test.describe suite for Scenario Outline
+  /**
+     * Generate test.describe suite for Scenario Outline
      */
   getOutlineSuite(scenario, parent) {
     const node = new _testNode.TestNode(scenario, parent);
-    if (node.isSkipped()) {
-      return this.formatter.suite(node, []);
-    }
     const lines = [];
     let exampleIndex = 0;
     scenario.examples.forEach(examples => {
-      const titleFormat = this.getExamplesTitleFormat(scenario, examples);
+      const titleFormat = this.getExamplesTitleFormat(examples);
       examples.tableBody.forEach(exampleRow => {
         const testTitle = this.getOutlineTestTitle(titleFormat, examples, exampleRow, ++exampleIndex);
         const testLines = this.getOutlineTest(scenario, examples, exampleRow, testTitle, node);
@@ -177,8 +150,8 @@ class TestFile {
     });
     return this.formatter.suite(node, lines);
   }
-  /**
-     * Generate test from Examples row of Scenario Outline
+  /**
+     * Generate test from Examples row of Scenario Outline
      */
   // eslint-disable-next-line max-params
   getOutlineTest(scenario, examples, exampleRow, title, parent) {
@@ -186,12 +159,7 @@ class TestFile {
       name: title,
       tags: examples.tags
     }, parent);
-    if (this.skipByTagsExpression(node)) {
-      return [];
-    }
-    if (node.isSkipped()) {
-      return this.formatter.test(node, new Set(), []);
-    }
+    if (this.skipByTagsExpression(node)) return [];
     this.testNodes.push(node);
     const {
       fixtures,
@@ -199,17 +167,12 @@ class TestFile {
     } = this.getSteps(scenario, node.tags, exampleRow.id);
     return this.formatter.test(node, fixtures, lines);
   }
-  /**
-     * Generate test from Scenario
+  /**
+     * Generate test from Scenario
      */
   getTest(scenario, parent) {
     const node = new _testNode.TestNode(scenario, parent);
-    if (this.skipByTagsExpression(node)) {
-      return [];
-    }
-    if (node.isSkipped()) {
-      return this.formatter.test(node, new Set(), []);
-    }
+    if (this.skipByTagsExpression(node)) return [];
     this.testNodes.push(node);
     const {
       fixtures,
@@ -217,8 +180,8 @@ class TestFile {
     } = this.getSteps(scenario, node.tags);
     return this.formatter.test(node, fixtures, lines);
   }
-  /**
-     * Generate test steps
+  /**
+     * Generate test steps
      */
   getSteps(scenario, tags, outlineExampleRowId) {
     const testFixtureNames = new Set();
@@ -267,8 +230,8 @@ class TestFile {
       lines
     };
   }
-  /**
-     * Generate step for Given, When, Then
+  /**
+     * Generate step for Given, When, Then
      */
   // eslint-disable-next-line max-statements, complexity
   getStep(step, previousKeywordType, outlineExampleRowId) {
@@ -279,27 +242,26 @@ class TestFile {
       language: this.language,
       previousKeywordType
     });
-    const enKeyword = this.getStepEnglishKeyword(step);
+    let keyword = this.getStepKeyword(step);
     if (!stepDefinition) {
       this.undefinedSteps.push({
         keywordType,
         step,
         pickleStep
       });
-      return this.getMissingStep(enKeyword, keywordType, pickleStep);
+      return this.getMissingStep(keyword, keywordType, pickleStep);
     }
     // for cucumber-style stepConfig is undefined
     const stepConfig = (0, _stepConfig.getStepConfig)(stepDefinition);
-    if (stepConfig !== null && stepConfig !== void 0 && stepConfig.hasCustomTest) {
-      this.hasCustomTest = true;
-    }
-    if (!(0, _stepConfig.isPlaywrightStyle)(stepConfig)) {
-      this.hasCucumberStyle = true;
-    }
-    const fixtureNames = this.getStepFixtureNames(stepDefinition);
-    const line = (0, _stepConfig.isDecorator)(stepConfig) ? '' : this.formatter.step(enKeyword, pickleStep.text, pickleStep.argument, fixtureNames);
+    if (stepConfig !== null && stepConfig !== void 0 && stepConfig.hasCustomTest) this.hasCustomTest = true;
+    // for cucumber-style transform Given/When/Then -> Given_/When_/Then_
+    // to use own bddWorld (containing PW built-in fixtures)
+    if (!(0, _stepConfig.isPlaywrightStyle)(stepConfig)) keyword = `${keyword}_`;
+    // for decorator steps fixtureNames are defined later in second pass
+    const fixtureNames = (0, _stepConfig.isDecorator)(stepConfig) ? [] : (0, _createBdd.extractFixtureNames)(stepConfig === null || stepConfig === void 0 ? void 0 : stepConfig.fn);
+    const line = (0, _stepConfig.isDecorator)(stepConfig) ? '' : this.formatter.step(keyword, pickleStep.text, pickleStep.argument, fixtureNames);
     return {
-      keyword: enKeyword,
+      keyword,
       keywordType,
       fixtureNames,
       line,
@@ -326,29 +288,16 @@ class TestFile {
         const hasRowId = !outlineExampleRowId || astNodeIds.includes(outlineExampleRowId);
         return hasStepId && hasRowId;
       });
-      if (pickleStep) {
-        return pickleStep;
-      }
+      if (pickleStep) return pickleStep;
     }
     throw new Error(`Pickle step not found for step: ${step.text}`);
   }
-  getStepEnglishKeyword(step) {
-    const nativeKeyword = step.keyword.trim();
-    const enKeyword = nativeKeyword === '*' ? 'And' : this.getEnglishKeyword(nativeKeyword);
-    if (!enKeyword) {
-      throw new Error(`Keyword not found: ${nativeKeyword}`);
-    }
+  getStepKeyword(step) {
+    const origKeyword = step.keyword.trim();
+    const enKeyword = origKeyword === '*' ? 'And' : this.getEnglishKeyword(origKeyword);
+    if (!enKeyword) throw new Error(`Keyword not found: ${origKeyword}`);
     return enKeyword;
   }
-  getStepFixtureNames(stepDefinition) {
-    const stepConfig = (0, _stepConfig.getStepConfig)(stepDefinition);
-    if ((0, _stepConfig.isPlaywrightStyle)(stepConfig)) {
-      // for decorator steps fixtureNames are defined later in second pass
-      return (0, _stepConfig.isDecorator)(stepConfig) ? [] : (0, _fixtures.extractFixtureNames)(stepConfig.fn);
-    } else {
-      return (0, _fixtures.extractFixtureNamesFromFnBodyMemo)(stepDefinition.code);
-    }
-  }
   getDecoratorStep(step, testPoms) {
     const {
       keyword,
@@ -374,16 +323,11 @@ class TestFile {
     exampleRow.cells.forEach((cell, index) => {
       var _examples$tableHeader;
       const colName = (_examples$tableHeader = examples.tableHeader) === null || _examples$tableHeader === void 0 || (_examples$tableHeader = _examples$tableHeader.cells[index]) === null || _examples$tableHeader === void 0 ? void 0 : _examples$tableHeader.value;
-      if (colName) {
-        params[colName] = cell.value;
-      }
+      if (colName) params[colName] = cell.value;
     });
     return (0, _utils.template)(titleFormat, params);
   }
-  getExamplesTitleFormat(scenario, examples) {
-    return this.getExamplesTitleFormatFromComment(examples) || this.getExamplesTitleFormatFromScenarioName(scenario, examples) || this.config.examplesTitleFormat;
-  }
-  getExamplesTitleFormatFromComment(examples) {
+  getExamplesTitleFormat(examples) {
     var _comment$text;
     const {
       line
@@ -394,22 +338,12 @@ class TestFile {
     });
     const commentText = comment === null || comment === void 0 || (_comment$text = comment.text) === null || _comment$text === void 0 ? void 0 : _comment$text.trim();
     const prefix = '# title-format:';
-    return commentText !== null && commentText !== void 0 && commentText.startsWith(prefix) ? commentText.replace(prefix, '').trim() : '';
-  }
-  getExamplesTitleFormatFromScenarioName(scenario, examples) {
-    var _examples$tableHeader2;
-    const columnsInScenarioName = (0, _utils.extractTemplateParams)(scenario.name);
-    const hasColumnsFromExamples = columnsInScenarioName.length && ((_examples$tableHeader2 = examples.tableHeader) === null || _examples$tableHeader2 === void 0 || (_examples$tableHeader2 = _examples$tableHeader2.cells) === null || _examples$tableHeader2 === void 0 ? void 0 : _examples$tableHeader2.some(cell => {
-      return cell.value && columnsInScenarioName.includes(cell.value);
-    }));
-    return hasColumnsFromExamples ? scenario.name : '';
+    return commentText !== null && commentText !== void 0 && commentText.startsWith(prefix) ? commentText.replace(prefix, '').trim() : this.config.examplesTitleFormat;
   }
   skipByTagsExpression(node) {
+    var _this$options$tagsExp;
     // see: https://github.com/cucumber/tag-expressions/tree/main/javascript
-    const {
-      tagsExpression
-    } = this.options;
-    return tagsExpression && !tagsExpression.evaluate(node.tags);
+    return ((_this$options$tagsExp = this.options.tagsExpression) === null || _this$options$tagsExp === void 0 ? void 0 : _this$options$tagsExp.evaluate(node.tags)) === false;
   }
   isOutline(scenario) {
     const keyword = this.getEnglishKeyword(scenario.keyword);

package/build/bdd-framework/gen/testNode.js

@@ -5,9 +5,9 @@ Object.defineProperty(exports, "__esModule", {
 });
 exports.TestNode = void 0;
 var _utils = require("../utils");
-/**
- * Universal TestNode class of parent-child relations in test file structure.
- * Holds tags and titles path.
+/**
+ * Universal TestNode class of parent-child relations in test file structure.
+ * Holds tags and titles path.
  */
 
 const SPECIAL_TAGS = ['@only', '@skip', '@fixme'];
@@ -23,9 +23,6 @@ class TestNode {
     this.title = gherkinNode.name;
     this.titlePath = ((parent === null || parent === void 0 ? void 0 : parent.titlePath) || []).concat([this.title]);
   }
-  isSkipped() {
-    return this.flags.skip || this.flags.fixme;
-  }
   initOwnTags(gherkinNode) {
     const tagNames = (0, _utils.removeDuplicates)(getTagNames(gherkinNode.tags));
     tagNames.forEach(tag => {
@@ -36,20 +33,10 @@ class TestNode {
       }
     });
   }
-  // eslint-disable-next-line complexity
   setFlag(tag) {
-    // in case of several system tags, @only takes precendence
-    if (tag === '@only') {
-      this.flags = {
-        only: true
-      };
-    }
-    if (tag === '@skip' && !this.flags.only) {
-      this.flags.skip = true;
-    }
-    if (tag === '@fixme' && !this.flags.only) {
-      this.flags.fixme = true;
-    }
+    if (tag === '@only') this.flags.only = true;
+    if (tag === '@skip') this.flags.skip = true;
+    if (tag === '@fixme') this.flags.fixme = true;
   }
 }
 exports.TestNode = TestNode;

package/build/bdd-framework/gen/testPoms.js

@@ -5,29 +5,29 @@ Object.defineProperty(exports, "__esModule", {
 });
 exports.TestPoms = void 0;
 exports.buildFixtureTag = buildFixtureTag;
-var _class = require("../stepDefinitions/decorators/class");
+var _poms = require("../stepDefinitions/decorators/poms");
 var _exit = require("../utils/exit");
-/**
- * Track PomNodes used in the particular test.
- * To select correct fixture for decorator steps.
- *
- * Idea: try to use the deepest child fixture for parent steps.
- *
- * Example inheritance tree:
- *      A
- *     / \
- *    B   C
- *   / \   \
- *  D   E   F
- *
- * If test uses steps from classes A and D:
- * -> resolved fixture will be D, even for steps from A.
- *
- * If test uses steps from classes A, D and C:
- * -> error, b/c A has 2 possible fixtures.
- *
- * If test uses steps from classes A and C, but @fixture tag is D:
- * -> error, b/c A has 2 possible fixtures.
+/**
+ * Track PomNodes used in the particular test.
+ * To select correct fixture for decorator steps.
+ *
+ * Idea: try to use the deepest child fixture for parent steps.
+ *
+ * Example inheritance tree:
+ *      A
+ *     / \
+ *    B   C
+ *   / \   \
+ *  D   E   F
+ *
+ * If test uses steps from classes A and D:
+ * -> resolved fixture will be D, even for steps from A.
+ *
+ * If test uses steps from classes A, D and C:
+ * -> error, b/c A has 2 possible fixtures.
+ *
+ * If test uses steps from classes A and C, but @fixture tag is D:
+ * -> error, b/c A has 2 possible fixtures.
  */
 
 const FIXTURE_TAG_PREFIX = '@fixture:';
@@ -44,7 +44,7 @@ class TestPoms {
     });
   }
   addByFixtureName(fixtureName) {
-    const pomNode = (0, _class.getPomNodeByFixtureName)(fixtureName);
+    const pomNode = (0, _poms.getPomNodeByFixtureName)(fixtureName);
     this.addUsedPom(pomNode, {
       byTag: false
     });
@@ -52,35 +52,31 @@ class TestPoms {
   addByTag(tag) {
     const fixtureName = extractFixtureName(tag);
     if (fixtureName) {
-      const pomNode = (0, _class.getPomNodeByFixtureName)(fixtureName);
+      const pomNode = (0, _poms.getPomNodeByFixtureName)(fixtureName);
       this.addUsedPom(pomNode, {
         byTag: true
       });
     }
   }
-  /**
-     * Resolve all used pomNodes to fixtures.
-     * This is needed to handle @fixture: tagged pomNodes
-     * that does not have steps in the test, but should be considered.
+  /**
+     * Resolve all used pomNodes to fixtures.
+     * This is needed to handle @fixture: tagged pomNodes
+     * that does not have steps in the test, but should be considered.
      */
   resolveFixtures() {
     this.usedPoms.forEach((_, pomNode) => {
       this.getResolvedFixtures(pomNode);
     });
   }
-  /**
-     * Returns fixtures suitable for particular pomNode (actually for step)
+  /**
+     * Returns fixtures suitable for particular pomNode (actually for step)
      */
   getResolvedFixtures(pomNode) {
     const usedPom = this.usedPoms.get(pomNode);
-    if (usedPom !== null && usedPom !== void 0 && usedPom.fixtures) {
-      return usedPom.fixtures;
-    }
+    if (usedPom !== null && usedPom !== void 0 && usedPom.fixtures) return usedPom.fixtures;
     // Recursively resolve children fixtures, used in test.
     let childFixtures = [...pomNode.children].map(child => this.getResolvedFixtures(child)).flat();
-    if (!usedPom) {
-      return childFixtures;
-    }
+    if (!usedPom) return childFixtures;
     if (childFixtures.length) {
       this.verifyChildFixtures(pomNode, usedPom, childFixtures);
       usedPom.fixtures = childFixtures;
@@ -95,29 +91,23 @@ class TestPoms {
   addUsedPom(pomNode, {
     byTag
   }) {
-    if (!pomNode) {
-      return;
-    }
+    if (!pomNode) return;
     const usedPom = this.usedPoms.get(pomNode);
     if (usedPom) {
-      if (byTag && !usedPom.byTag) {
-        usedPom.byTag = true;
-      }
+      if (byTag && !usedPom.byTag) usedPom.byTag = true;
     } else {
       this.usedPoms.set(pomNode, {
         byTag
       });
     }
   }
-  /**
-     * For scenarios with @fixture:xxx tags verify that there are no steps from fixtures,
-     * deeper than xxx.
-     * @fixture:xxx tag provides maximum fixture that can be used in the scenario.
+  /**
+     * For scenarios with @fixture:xxx tags verify that there are no steps from fixtures,
+     * deeper than xxx.
+     * @fixture:xxx tag provides maximum fixture that can be used in the scenario.
      */
   verifyChildFixtures(pomNode, usedPom, childFixtures) {
-    if (!usedPom.byTag) {
-      return;
-    }
+    if (!usedPom.byTag) return;
     const childFixturesBySteps = childFixtures.filter(f => !f.byTag);
     if (childFixturesBySteps.length) {
       (0, _exit.exit)(`Scenario "${this.title}" contains ${childFixturesBySteps.length} step(s)`, `not compatible with required fixture "${pomNode.fixtureName}"`);

package/build/bdd-framework/playwright/fixtureParameterNames.js

@@ -4,31 +4,23 @@ Object.defineProperty(exports, "__esModule", {
   value: true
 });
 exports.fixtureParameterNames = fixtureParameterNames;
-/**
- * Extracted from playwright.
- * https://github.com/microsoft/playwright/blob/main/packages/playwright-test/src/common/fixtures.ts#L226
+/**
+ * Extracted from playwright.
+ * https://github.com/microsoft/playwright/blob/main/packages/playwright-test/src/common/fixtures.ts#L226
  */
 /* eslint-disable max-statements, complexity, max-len, max-depth */
 const signatureSymbol = Symbol('signature');
 function fixtureParameterNames(fn) {
-  if (typeof fn !== 'function') {
-    return [];
-  }
-  if (!fn[signatureSymbol]) {
-    fn[signatureSymbol] = innerFixtureParameterNames(fn);
-  }
+  if (typeof fn !== 'function') return [];
+  if (!fn[signatureSymbol]) fn[signatureSymbol] = innerFixtureParameterNames(fn);
   return fn[signatureSymbol];
 }
 function innerFixtureParameterNames(fn) {
   const text = filterOutComments(fn.toString());
   const match = text.match(/(?:async)?(?:\s+function)?[^(]*\(([^)]*)/);
-  if (!match) {
-    return [];
-  }
+  if (!match) return [];
   const trimmedParams = match[1].trim();
-  if (!trimmedParams) {
-    return [];
-  }
+  if (!trimmedParams) return [];
   const [firstParam] = splitByComma(trimmedParams);
   if (firstParam[0] !== '{' || firstParam[firstParam.length - 1] !== '}') {
     throw new Error('First argument must use the object destructuring pattern: ' + firstParam + ' ' + fn.toString());
@@ -48,13 +40,9 @@ function filterOutComments(s) {
   let commentState = 'none';
   for (let i = 0; i < s.length; ++i) {
     if (commentState === 'singleline') {
-      if (s[i] === '\n') {
-        commentState = 'none';
-      }
+      if (s[i] === '\n') commentState = 'none';
     } else if (commentState === 'multiline') {
-      if (s[i - 1] === '*' && s[i] === '/') {
-        commentState = 'none';
-      }
+      if (s[i - 1] === '*' && s[i] === '/') commentState = 'none';
     } else if (commentState === 'none') {
       if (s[i] === '/' && s[i + 1] === '/') {
         commentState = 'singleline';
@@ -79,15 +67,11 @@ function splitByComma(s) {
       stack.pop();
     } else if (!stack.length && s[i] === ',') {
       const token = s.substring(start, i).trim();
-      if (token) {
-        result.push(token);
-      }
+      if (token) result.push(token);
       start = i + 1;
     }
   }
   const lastToken = s.substring(start).trim();
-  if (lastToken) {
-    result.push(lastToken);
-  }
+  if (lastToken) result.push(lastToken);
   return result;
 }