@zohodesk/testinglibrary 0.1.9-exp-actors → 0.2.0

package/build/bdd-framework/reporter/cucumber/messagesBuilder/AttachmentMapper.js

@@ -5,8 +5,8 @@ Object.defineProperty(exports, "__esModule", {
 });
 exports.AttachmentMapper = void 0;
 var _AutofillMap = require("../../../utils/AutofillMap");
-var _pwUtils = require("./pwUtils");
-var _bddDataAttachment = require("../../../run/bddDataAttachment");
+var _pwStepUtils = require("./pwStepUtils");
+var _stripAnsiEscapes = require("../../../utils/stripAnsiEscapes");
 class AttachmentMapper {
   result;
   stepAttachments = new _AutofillMap.AutofillMap();
@@ -20,12 +20,14 @@ class AttachmentMapper {
   }
   mapAttachments() {
     const allAttachments = this.result.attachments.slice();
-    const attachmentSteps = (0, _pwUtils.collectStepsWithCategory)(this.result, 'attach');
+    const attachmentSteps = (0, _pwStepUtils.collectStepsWithCategory)(this.result, 'attach');
     attachmentSteps.forEach(attachmentStep => {
       this.mapAttachment(attachmentStep, allAttachments);
     });
     this.unusedAttachments.push(...allAttachments);
     this.mapUnusedAttachments();
+    this.mapStdoutAttachments('stdout');
+    this.mapStdoutAttachments('stderr');
   }
   mapAttachment(attachmentStep, allAttachments) {
     const index = allAttachments.findIndex(a => getAttachmentStepTitle(a.name) === attachmentStep.title);
@@ -33,9 +35,6 @@ class AttachmentMapper {
       throw new Error(`Attachment not found for step: ${attachmentStep.title}`);
     }
     const [foundAttachment] = allAttachments.splice(index, 1);
-    if ((0, _bddDataAttachment.isBddDataAttachment)(foundAttachment)) {
-      return;
-    }
     const parentStep = attachmentStep.parent;
     // step.parent is empty:
     // - in PW = 1.34 for screenshot attachment
@@ -49,13 +48,32 @@ class AttachmentMapper {
       return;
     }
     // map unused attachments to the 'After Hooks' step
-    const afterHooksRoot = (0, _pwUtils.getHooksRootStep)(this.result, 'after');
-    if (!afterHooksRoot) {
-      throw new Error(`Can not find after hooks root to attach unused attachments.`);
-    }
+    const afterHooksRoot = this.getAfterHooksRoot();
     const stepAttachments = this.stepAttachments.getOrCreate(afterHooksRoot, () => []);
     stepAttachments.push(...this.unusedAttachments);
   }
+  mapStdoutAttachments(name) {
+    var _this$result$name;
+    // map stdout / stderr to the 'After Hooks' step
+    if (!((_this$result$name = this.result[name]) !== null && _this$result$name !== void 0 && _this$result$name.length)) {
+      return;
+    }
+    const body = this.result[name].map(s => typeof s === 'string' ? (0, _stripAnsiEscapes.stripAnsiEscapes)(s) : s).join('');
+    const afterHooksRoot = this.getAfterHooksRoot();
+    const stepAttachments = this.stepAttachments.getOrCreate(afterHooksRoot, () => []);
+    stepAttachments.push({
+      name,
+      contentType: 'text/plain',
+      body: Buffer.from(body)
+    });
+  }
+  getAfterHooksRoot() {
+    const afterHooksRoot = (0, _pwStepUtils.getHooksRootPwStep)(this.result, 'after');
+    if (!afterHooksRoot) {
+      throw new Error(`Can not find after hooks root.`);
+    }
+    return afterHooksRoot;
+  }
 }
 // See: https://github.com/microsoft/playwright/blob/main/packages/playwright/src/worker/testInfo.ts#L413
 exports.AttachmentMapper = AttachmentMapper;

package/build/bdd-framework/reporter/cucumber/messagesBuilder/Builder.js

@@ -43,10 +43,16 @@ class MessagesBuilder {
     this.onEndPromise = new Promise(resolve => this.onEndPromiseResolve = resolve);
   }
   onTestEnd(test, result) {
-    this.onTestEnds.push({
-      test,
-      result
-    });
+    // For skipped tests Playwright doesn't run fixtures
+    // and we don't have bddData attachment -> don't know feature uri.
+    // Don't add such test run to report.
+    if (test.expectedStatus === 'skipped') {
+      return;
+    }
+    // Important to create TestCaseRun here,
+    // b/c test properties can change after retries (e.g. annotations)
+    const testCaseRun = new _TestCaseRun.TestCaseRun(test, result, this.hooks);
+    this.testCaseRuns.push(testCaseRun);
   }
   onEnd(fullResult) {
     this.fullResult = fullResult;
@@ -66,7 +72,7 @@ class MessagesBuilder {
   async doBuildMessages() {
     await this.onEndPromise;
     // order here is important
-    this.createTestCaseRuns();
+    // this.createTestCaseRuns();
     await this.loadFeatures();
     this.createTestCases();
     this.addMeta();
@@ -100,21 +106,16 @@ class MessagesBuilder {
   async loadFeatures() {
     await this.gherkinDocuments.load(this.testCaseRuns);
   }
-  createTestCaseRuns() {
-    this.onTestEnds.forEach(({
-      test,
-      result
-    }) => {
-      // For skipped tests Playwright doesn't run fixtures
-      // and we don't have bddData attachment -> don't know feature uri.
-      // Don't add such test run to report.
-      if (test.expectedStatus === 'skipped') {
-        return;
-      }
-      const testCaseRun = new _TestCaseRun.TestCaseRun(test, result, this.hooks);
-      this.testCaseRuns.push(testCaseRun);
-    });
-  }
+  // private createTestCaseRuns() {
+  //   this.onTestEnds.forEach(({ test, result }) => {
+  //     // For skipped tests Playwright doesn't run fixtures
+  //     // and we don't have bddData attachment -> don't know feature uri.
+  //     // Don't add such test run to report.
+  //     if (test.expectedStatus === 'skipped') return;
+  //     const testCaseRun = new TestCaseRun(test, result, this.hooks);
+  //     this.testCaseRuns.push(testCaseRun);
+  //   });
+  // }
   addMeta() {
     this.report.meta = new _Meta.Meta().buildMessage();
   }

package/build/bdd-framework/reporter/cucumber/messagesBuilder/Hook.js

@@ -60,10 +60,10 @@ class Hook {
     const uri = file ? _path.default.relative((0, _configDir.getPlaywrightConfigDir)(), file) : undefined;
     return {
       uri,
-      location: {
-        line: line || 0,
+      location: line ? {
+        line,
         column
-      }
+      } : undefined
     };
   }
 }

package/build/bdd-framework/reporter/cucumber/messagesBuilder/TestCaseRun.js

@@ -7,11 +7,11 @@ exports.TestCaseRun = void 0;
 var _utils = require("../../../utils");
 var _TestStepRun = require("./TestStepRun");
 var _timing = require("./timing");
-var _pwUtils = require("./pwUtils");
-var _bddDataAttachment = require("../../../run/bddDataAttachment");
+var _pwStepUtils = require("./pwStepUtils");
 var _AttachmentMapper = require("./AttachmentMapper");
 var _TestCaseRunHooks = require("./TestCaseRunHooks");
 var _Projects = require("./Projects");
+var _bddData = require("../../../run/bddData");
 class TestCaseRun {
   test;
   result;
@@ -21,6 +21,10 @@ class TestCaseRun {
   testCase;
   attachmentMapper;
   projectInfo;
+  // collect steps with error and show only these errors in report.
+  // it allows to not show the same error on parent steps
+  errorSteps = new Set();
+  timeoutedStep;
   executedBeforeHooks;
   executedAfterHooks;
   executedSteps;
@@ -30,7 +34,7 @@ class TestCaseRun {
     this.result = result;
     this.hooks = hooks;
     this.id = this.generateTestRunId();
-    this.bddData = this.getBddData();
+    this.bddData = this.extractBddData();
     this.projectInfo = (0, _Projects.getProjectInfo)(this.test);
     this.attachmentMapper = new _AttachmentMapper.AttachmentMapper(this.result);
     this.executedSteps = this.fillExecutedSteps();
@@ -43,20 +47,30 @@ class TestCaseRun {
     }
     return this.testCase;
   }
+  isTimeouted() {
+    return this.result.status === 'timedOut';
+  }
   generateTestRunId() {
-    return `${this.test.id}-run-${this.result.retry}`;
+    return `${this.test.id}-attempt-${this.result.retry}`;
   }
-  getBddData() {
-    const bddData = (0, _bddDataAttachment.getBddDataFromTestResult)(this.result);
+  extractBddData() {
+    const {
+      bddData,
+      annotationIndex
+    } = (0, _bddData.getBddDataFromTest)(this.test);
     if (!bddData) {
-      throw new Error([`BDD data attachment is not found for test: ${this.test.title}`, `Did you set enrichReporterData: true in the Playwright config?`, ''].join('\n'));
+      throw new Error(`__bddData annotation is not found for test "${this.test.title}".`);
     }
+    // remove __bddData annotation from test (mutate)
+    this.test.annotations.splice(annotationIndex, 1);
     return bddData;
   }
   fillExecutedSteps() {
-    const possiblePwSteps = this.getPossiblePlaywrightSteps();
+    const possiblePwSteps = this.getPossiblePlaywrightBddSteps();
     return this.bddData.steps.map(bddDataStep => {
       const pwStep = this.findPlaywrightStep(possiblePwSteps, bddDataStep);
+      this.registerErrorStep(pwStep);
+      this.registerTimeoutedStep(pwStep);
       return {
         bddDataStep,
         pwStep
@@ -66,6 +80,23 @@ class TestCaseRun {
   fillExecutedHooks(hookType) {
     return new _TestCaseRunHooks.TestCaseRunHooks(this, hookType).fill(this.executedSteps);
   }
+  registerErrorStep(pwStep) {
+    if (pwStep !== null && pwStep !== void 0 && pwStep.error) {
+      this.errorSteps.add(pwStep);
+    }
+  }
+  // eslint-disable-next-line complexity
+  registerTimeoutedStep(pwStep) {
+    if (!pwStep || !this.isTimeouted() || this.timeoutedStep) {
+      return;
+    }
+    const {
+      error
+    } = pwStep;
+    if ((0, _pwStepUtils.isUnknownDuration)(pwStep) || this.result.errors.some(e => e.message === (error === null || error === void 0 ? void 0 : error.message))) {
+      this.timeoutedStep = pwStep;
+    }
+  }
   buildMessages() {
     return [this.buildTestCaseStarted(), ...this.executedBeforeHooks.buildMessages(), ...this.buildStepRuns(), ...this.executedAfterHooks.buildMessages(), this.buildTestCaseFinished()];
   }
@@ -108,19 +139,16 @@ class TestCaseRun {
     };
   }
   findPlaywrightStep(possiblePwSteps, bddDataStep) {
-    const pwStep = possiblePwSteps.find(pwStep => {
+    return possiblePwSteps.find(pwStep => {
       return pwStep.location && (0, _utils.stringifyLocation)(pwStep.location) === bddDataStep.pwStepLocation;
     });
-    if (!pwStep) {
-      throw new Error('Playwright step not found for bdd step');
-    }
-    return pwStep;
   }
-  getPossiblePlaywrightSteps() {
-    const beforeHooksRoot = (0, _pwUtils.getHooksRootStep)(this.result, 'before');
-    const bgSteps = (0, _pwUtils.collectStepsWithCategory)(beforeHooksRoot, 'test.step');
-    const topLevelSteps = this.result.steps.filter(step => step.category === 'test.step');
-    return [...bgSteps, ...topLevelSteps];
+  getPossiblePlaywrightBddSteps() {
+    // Before we collected only top-level steps and steps from before hooks (as they are background)
+    // But it's more reliable to just collect all test.step items b/c some Playwright versions
+    // move steps to fixtures (see: https://github.com/microsoft/playwright/issues/30075)
+    // Collecting all test.step items should be ok, as later we anyway map them by location.
+    return (0, _pwStepUtils.collectStepsWithCategory)(this.result, 'test.step');
   }
 }
 exports.TestCaseRun = TestCaseRun;

package/build/bdd-framework/reporter/cucumber/messagesBuilder/TestCaseRunHooks.js

@@ -5,12 +5,12 @@ Object.defineProperty(exports, "__esModule", {
 });
 exports.TestCaseRunHooks = void 0;
 var _Hook = require("./Hook");
-var _pwUtils = require("./pwUtils");
+var _pwStepUtils = require("./pwStepUtils");
 var _TestStepRun = require("./TestStepRun");
 class TestCaseRunHooks {
   testCaseRun;
   hookType;
-  rootStep;
+  rootPwStep;
   candidateSteps = [];
   hookSteps = new Set();
   executedHooks = new Map();
@@ -22,9 +22,10 @@ class TestCaseRunHooks {
     this.setRootStep();
     this.setCandidateSteps();
     this.addStepsWithName();
-    this.addStepsWithAttachment();
     this.addStepWithError();
-    this.excludeBackgroundSteps(mainSteps);
+    this.addStepWithTimeout();
+    this.addStepsWithAttachment();
+    this.excludeMainSteps(mainSteps);
     this.setExecutedHooks();
     return this;
   }
@@ -33,7 +34,7 @@ class TestCaseRunHooks {
     this.testCaseRun.getTestCase().getHooks(this.hookType).forEach(hookInfo => {
       const executedHook = this.executedHooks.get(hookInfo.hook.internalId);
       // todo: if pwStep is not found in this.executedBeforeHooks,
-      // it means that this hook comes from another run of this test case.
+      // it means that this hook comes from another attempt of this test case.
       // We can stil try to find it in test result, as otherwise it will be marked as skipped,
       // but actually it was executed.
       const testStepRun = new _TestStepRun.TestStepRun(this.testCaseRun, hookInfo.testStep, executedHook === null || executedHook === void 0 ? void 0 : executedHook.pwStep);
@@ -42,13 +43,13 @@ class TestCaseRunHooks {
     return messages;
   }
   setRootStep() {
-    this.rootStep = (0, _pwUtils.getHooksRootStep)(this.testCaseRun.result, this.hookType);
+    this.rootPwStep = (0, _pwStepUtils.getHooksRootPwStep)(this.testCaseRun.result, this.hookType);
   }
   setCandidateSteps() {
-    if (this.rootStep) {
-      this.candidateSteps.push(this.rootStep);
+    if (this.rootPwStep) {
+      this.candidateSteps.push(this.rootPwStep);
     }
-    this.candidateSteps.push(...(0, _pwUtils.collectStepsDfs)(this.rootStep));
+    this.candidateSteps.push(...(0, _pwStepUtils.collectStepsDfs)(this.rootPwStep));
   }
   addStepsWithName() {
     this.candidateSteps.forEach(pwStep => {
@@ -68,25 +69,45 @@ class TestCaseRunHooks {
     });
   }
   addStepWithError() {
-    const stepWithError = (0, _pwUtils.findDeepestErrorStep)(this.rootStep);
+    const stepWithError = (0, _pwStepUtils.findDeepestStepWithError)(this.rootPwStep);
     if (stepWithError) {
       this.hookSteps.add(stepWithError);
       // in Playwright error is inherited by all parent steps,
       // but we want to show it once (in the deepest step)
-      this.hookSteps.forEach(step => {
-        if (step !== stepWithError) {
-          delete step.error;
-        }
-      });
+      this.testCaseRun.registerErrorStep(stepWithError);
+      this.testCaseRun.registerTimeoutedStep(stepWithError);
+    }
+  }
+  addStepWithTimeout() {
+    if (!this.testCaseRun.isTimeouted()) {
+      return;
+    }
+    if (this.testCaseRun.timeoutedStep) {
+      return;
+    }
+    const timeoutedStep = this.hookType === 'before' ?
+    // Timeouted steps have duration = -1 in PW <= 1.39 and no error field.
+    // In PW > 1.39 timeouted steps have '.error' populated
+    (0, _pwStepUtils.findDeepestStepWithUnknownDuration)(this.rootPwStep) :
+    // Timeouted after hooks don't have duration = -1,
+    // so there is no way to find which exactly fixture timed out.
+    // We mark root 'After Hooks' step as timeouted.
+    this.rootPwStep;
+    if (timeoutedStep) {
+      this.hookSteps.add(timeoutedStep);
+      this.testCaseRun.timeoutedStep = timeoutedStep;
     }
   }
-  excludeBackgroundSteps(mainSteps) {
-    // exclude background steps, b/c they are in pickle, not in hooks.
+  excludeMainSteps(mainSteps) {
+    // - exclude background steps, b/c they are in pickle and should not in hooks.
+    // - exclude other test.step items that are bdd steps and should not be in hooks.
     // Important to run this fn after this.fillExecutedSteps()
     // as we assume steps are already populated
-    if (this.hookType === 'before') {
-      mainSteps.forEach(stepInfo => this.hookSteps.delete(stepInfo.pwStep));
-    }
+    mainSteps.forEach(stepInfo => {
+      if (stepInfo.pwStep) {
+        this.hookSteps.delete(stepInfo.pwStep);
+      }
+    });
   }
   setExecutedHooks() {
     this.hookSteps.forEach(pwStep => {

package/build/bdd-framework/reporter/cucumber/messagesBuilder/TestStepAttachments.js

@@ -7,6 +7,7 @@ Object.defineProperty(exports, "__esModule", {
 exports.TestStepAttachments = void 0;
 var _fs = _interopRequireDefault(require("fs"));
 var messages = _interopRequireWildcard(require("@cucumber/messages"));
+var _path = _interopRequireDefault(require("path"));
 function _getRequireWildcardCache(e) { if ("function" != typeof WeakMap) return null; var r = new WeakMap(), t = new WeakMap(); return (_getRequireWildcardCache = function (e) { return e ? t : r; })(e); }
 function _interopRequireWildcard(e, r) { if (!r && e && e.__esModule) return e; if (null === e || "object" != typeof e && "function" != typeof e) return { default: e }; var t = _getRequireWildcardCache(r); if (t && t.has(e)) return t.get(e); var n = { __proto__: null }, a = Object.defineProperty && Object.getOwnPropertyDescriptor; for (var u in e) if ("default" !== u && Object.prototype.hasOwnProperty.call(e, u)) { var i = a ? Object.getOwnPropertyDescriptor(e, u) : null; i && (i.get || i.set) ? Object.defineProperty(n, u, i) : n[u] = e[u]; } return n.default = e, t && t.set(e, n), n; }
 /**
@@ -44,7 +45,23 @@ class TestStepAttachments {
     };
   }
   getAttachmentBodyBase64(pwAttachment) {
-    return pwAttachment.path ? _fs.default.readFileSync(pwAttachment.path, 'base64') : pwAttachment.body.toString('base64');
+    if (pwAttachment.path) {
+      if (_fs.default.existsSync(pwAttachment.path)) {
+        return _fs.default.readFileSync(pwAttachment.path, 'base64');
+      }
+      throw createMissingAttachmentError(pwAttachment.path);
+    }
+    if (pwAttachment.body) {
+      return pwAttachment.body.toString('base64');
+    }
+    throw new Error(`Playwright attachment without path and body`);
   }
 }
-exports.TestStepAttachments = TestStepAttachments;
+exports.TestStepAttachments = TestStepAttachments;
+function createMissingAttachmentError(attachmentPath) {
+  const attachmentDir = _path.default.join(_path.default.dirname(attachmentPath));
+  const attachmentDirExists = _fs.default.existsSync(attachmentDir);
+  const files = attachmentDirExists ? _fs.default.readdirSync(attachmentDir) : [];
+  const errorMsg = [`Attachment file is not found:`, attachmentPath, `Attachment dir ${attachmentDirExists ? 'exists' : 'does not exist'}.`, ...(attachmentDirExists ? [`Available files (${files.length}):`, ...files] : []), ''].join('\n');
+  return new Error(errorMsg);
+}

package/build/bdd-framework/reporter/cucumber/messagesBuilder/TestStepRun.js

@@ -25,6 +25,9 @@ class TestStepRun {
     // prettier-ignore
     ...stepAttachments.buildMessages(), this.buildTestStepFinished()];
   }
+  isHook() {
+    return Boolean(this.testStep.hookId);
+  }
   wasExecuted() {
     return Boolean(this.pwStep);
   }
@@ -45,16 +48,17 @@ class TestStepRun {
     };
   }
   buildTestStepFinished() {
-    var _this$pwStep;
-    const error = (_this$pwStep = this.pwStep) === null || _this$pwStep === void 0 ? void 0 : _this$pwStep.error;
+    const error = this.getStepError();
     const testStepFinished = {
       testCaseStartedId: this.testCaseRun.id,
       testStepId: this.testStep.id,
       testStepResult: {
         duration: messages.TimeConversion.millisecondsToDuration(this.duration),
         status: this.getStatus(error),
-        message: error ? formatError(error) : undefined,
-        exception: error ? this.buildException(error) : undefined
+        // 'message' is deprecated since cucumber 10.4 in favor of 'exception' field
+        // See: https://github.com/cucumber/react-components/pull/345
+        message: error ? buildErrorMessage(error) : undefined,
+        exception: error ? buildException(error) : undefined
       },
       timestamp: (0, _timing.toCucumberTimestamp)(this.startTime.getTime() + this.duration)
     };
@@ -62,27 +66,49 @@ class TestStepRun {
       testStepFinished
     };
   }
-  buildException(error) {
-    return {
-      type: 'Error',
-      message: error.message ? (0, _stripAnsiEscapes.stripAnsiEscapes)(error.message) : undefined,
-      stackTrace: error.stack ? (0, _stripAnsiEscapes.stripAnsiEscapes)(error.stack) : undefined
-      // Use type casting b/c older versions of @cucumber/messages don't have 'stackTrace' field
-      // todo: add direct dependency on @cucumber/messages
-    };
-  }
   getStatus(error) {
     switch (true) {
-      case !this.wasExecuted():
-        return messages.TestStepResultStatus.SKIPPED;
       case Boolean(error):
         return messages.TestStepResultStatus.FAILED;
+      // For hooks that were not executted we return PASSED, not SKIPPED.
+      // Because these hooks can be from another run attempt of this testCase.
+      // If marked as skipped, the whole run is marked as skipped in reporter.
+      case !this.isHook() && !this.wasExecuted():
+        return messages.TestStepResultStatus.SKIPPED;
       default:
         return messages.TestStepResultStatus.PASSED;
     }
   }
+  // eslint-disable-next-line complexity
+  getStepError() {
+    if (!this.pwStep) {
+      return;
+    }
+    if (this.testCaseRun.errorSteps.has(this.pwStep)) {
+      return this.pwStep.error;
+    }
+    if (this.testCaseRun.isTimeouted() && this.pwStep === this.testCaseRun.timeoutedStep) {
+      var _this$testCaseRun$res;
+      return {
+        message: (_this$testCaseRun$res = this.testCaseRun.result.errors) === null || _this$testCaseRun$res === void 0 ? void 0 : _this$testCaseRun$res.map(e => e.message).join('\n\n')
+      };
+    }
+  }
 }
 exports.TestStepRun = TestStepRun;
-function formatError(error) {
+function buildErrorMessage(error) {
   return (0, _stripAnsiEscapes.stripAnsiEscapes)([error.message, error.snippet].filter(Boolean).join('\n'));
+}
+function buildException(error) {
+  return {
+    type: 'Error',
+    message: buildErrorMessage(error),
+    // todo: extract only trace?
+    stackTrace: error.stack ? extractStackTrace((0, _stripAnsiEscapes.stripAnsiEscapes)(error.stack)) : undefined
+    // Use type casting b/c older versions of @cucumber/messages don't have 'stackTrace' field
+    // todo: add direct dependency on @cucumber/messages
+  };
+}
+function extractStackTrace(errorStack) {
+  return errorStack.split('\n').filter(line => line.match(/^\s+at .*/)).join('\n');
 }

package/build/bdd-framework/reporter/cucumber/messagesBuilder/{pwUtils.js → pwStepUtils.js}

@@ -5,37 +5,53 @@ Object.defineProperty(exports, "__esModule", {
 });
 exports.collectStepsDfs = collectStepsDfs;
 exports.collectStepsWithCategory = collectStepsWithCategory;
-exports.findDeepestErrorStep = findDeepestErrorStep;
-exports.getHooksRootStep = getHooksRootStep;
+exports.findDeepestStepWithError = findDeepestStepWithError;
+exports.findDeepestStepWithUnknownDuration = findDeepestStepWithUnknownDuration;
+exports.getHooksRootPwStep = getHooksRootPwStep;
+exports.isUnknownDuration = isUnknownDuration;
 /**
  * Utility functions for filtering Playwright steps.
  */
-// Playwright step categoires, that can be mapped to testStep / hook in Cucumber report
+// Playwright step categoires, that can be mapped to testStep / hook in Cucumber messages
 const MEANINGFUL_STEP_CATEGORIES = ['hook', 'fixture', 'test.step'];
 function collectStepsWithCategory(parent, category) {
   const categories = Array.isArray(category) ? category : [category];
   const steps = collectStepsDfs(parent);
   return steps.filter(step => categories.includes(step.category));
 }
-function getHooksRootStep(result, type) {
+function getHooksRootPwStep(result, type) {
   const rootStepTitle = type === 'before' ? 'Before Hooks' : 'After Hooks';
   return result.steps.find(step => step.category === 'hook' && step.title === rootStepTitle);
 }
+function findDeepestStepWithError(root) {
+  if (!root) {
+    return;
+  }
+  return findDeepestStepWith(root, pwStep => {
+    return Boolean(pwStep.error) && MEANINGFUL_STEP_CATEGORIES.includes(pwStep.category);
+  });
+}
+function findDeepestStepWithUnknownDuration(root) {
+  if (!root) {
+    return;
+  }
+  return findDeepestStepWith(root, pwStep => {
+    return isUnknownDuration(pwStep) && MEANINGFUL_STEP_CATEGORIES.includes(pwStep.category);
+  });
+}
 /**
- * Drills down to the deepest error step.
+ * Finds the deepest step that satisfies predicate function.
  */
-function findDeepestErrorStep(root) {
-  let errorStep = root !== null && root !== void 0 && root.error ? root : null;
-  while (errorStep) {
-    const nextErrorStep = errorStep.steps.find(step => {
-      return step.error && MEANINGFUL_STEP_CATEGORIES.includes(step.category);
-    });
-    if (!nextErrorStep) {
+function findDeepestStepWith(root, predicate) {
+  let currentStep = predicate(root) ? root : undefined;
+  while (currentStep) {
+    const nextStep = currentStep.steps.find(pwStep => predicate(pwStep));
+    if (!nextStep) {
       break;
     }
-    errorStep = nextErrorStep;
+    currentStep = nextStep;
   }
-  return errorStep;
+  return currentStep;
 }
 /**
  * Returns all steps in DFS order.
@@ -48,4 +64,7 @@ function collectStepsDfs(parent) {
     res.push(...collectStepsDfs(step));
     return res;
   }, [])) || [];
+}
+function isUnknownDuration(pwStep) {
+  return pwStep.duration === -1;
 }

package/build/bdd-framework/run/StepInvoker.js

@@ -7,7 +7,7 @@ exports.StepInvoker = void 0;
 var _loadSteps = require("../cucumber/loadSteps");
 var _getLocationInFile = require("../playwright/getLocationInFile");
 var _testTypeImpl = require("../playwright/testTypeImpl");
-var _defineStep = require("../stepDefinitions/defineStep");
+var _defineStep = require("../steps/defineStep");
 var _lang = require("../config/lang");
 /**
  * Class to invoke step in playwright runner.
@@ -25,17 +25,18 @@ class StepInvoker {
      * Invokes particular step.
      * See: https://github.com/cucumber/cucumber-js/blob/main/src/runtime/test_case_runner.ts#L299
      */
-  async invoke(text, argument, stepFixtures) {
+  async invoke(stepText, argument, stepFixtures) {
+    var _this$world$$internal;
     this.world.$internal.currentStepFixtures = stepFixtures || {};
-    const stepDefinition = this.getStepDefinition(text);
+    const stepDefinition = this.getStepDefinition(stepText);
     // Get location of step call in generated test file.
     // This call must be exactly here to have correct call stack (before async calls)
     const location = (0, _getLocationInFile.getLocationInFile)(this.world.testInfo.file);
-    const stepTitle = this.getStepTitle(text);
+    const stepTitle = this.getStepTitle(stepText);
     const code = (0, _defineStep.getStepCode)(stepDefinition);
-    const parameters = await this.getStepParameters(stepDefinition, text, argument || undefined);
-    this.world.$internal.bddData.registerStep(stepDefinition, text, location);
-    return (0, _testTypeImpl.runStepWithCustomLocation)(this.world.test, stepTitle, location, () => code.apply(this.world, parameters));
+    const parameters = await this.getStepParameters(stepDefinition, stepText, argument || undefined);
+    (_this$world$$internal = this.world.$internal.bddDataManager) === null || _this$world$$internal === void 0 || _this$world$$internal.registerStep(stepDefinition, stepText, location);
+    await (0, _testTypeImpl.runStepWithCustomLocation)(this.world.test, stepTitle, location, () => code.apply(this.world, parameters));
   }
   getStepDefinition(text) {
     const stepDefinition = (0, _loadSteps.findStepDefinition)(this.world.options.supportCodeLibrary, text, this.world.testInfo.file);