plum117 · Oct 21, 2026 · Oct 21, 2026 · Oct 21, 2026 · Oct 21, 2026 · Oct 21, 2026
@@ -6,7 +6,7 @@ import path from 'node:path'
 import logger from '@wdio/logger'
 
 import type { Payload } from './types.js'
-import { convertStatus, getAmbiguousStepMessage } from './utils.js'
+import { getAmbiguousStepMessage, getFeatureStack, getStepResult } from './utils.js'
 
 const log = logger('CucumberFormatter')
 
@@ -24,7 +24,6 @@ import type {
     TestStepStarted,
     TestStepFinished,
     TestStepResult,
-    TestCaseFinished,
     Hook,
     StepDefinition
 } from '@cucumber/messages'
@@ -223,53 +222,21 @@ export default class CucumberFormatter extends Formatter {
         step: PickleStep | undefined,
         result: TestStepResult
     ) {
-        let state = convertStatus(result.status)
-        let error = result.message ? new Error(result.message) : undefined
         const title = step ? step?.text : this.getTitle(scenario)
-        let titleSuffix = ''
-
-        if (result.status === Status.UNDEFINED) {
-            if (this.ignoreUndefinedDefinitions) {
-                /**
-                 * mark test as pending
-                 */
-                state = 'pending'
-                titleSuffix = ' (undefined step)'
-            } else {
-                /**
-                 * mark test as failed
-                 */
-                state = 'fail'
-                this.failedCount++
-
-                const err = new Error(
-                    (step ? `Step "${title}" is not defined. ` : `Scenario ${title} has undefined steps. `) +
-                    'You can ignore this error by setting cucumberOpts.ignoreUndefinedDefinitions as true.'
-                )
-
-                err.stack = `${err.message}\n${this.getFeatureStack(uri, feature, scenario, step)}`
-                error = err
+        const { state, error, pendingReason, titleSuffix, failed } = getStepResult(
+            result as TestStepResult & { willBeRetried?: boolean },
+            {
+                title,
+                isStep: Boolean(step),
+                ignoreUndefinedDefinitions: this.ignoreUndefinedDefinitions,
+                failAmbiguousDefinitions: this.failAmbiguousDefinitions,
+                featureStack: () => getFeatureStack(uri, feature, scenario, step),
             }
-        } else if (result.status === Status.FAILED && !(result as unknown as TestCaseFinished).willBeRetried) {
-            state = 'fail'
-            this.failedCount++
-            error = new Error(result.message?.split('\n')[0])
-            error.stack = result.message as string
-        } else if (result.status === Status.AMBIGUOUS && this.failAmbiguousDefinitions) {
-            state = 'fail'
+        )
+        if (failed) {
             this.failedCount++
-            const message = result.message || (
-                (step
-                    ? `Step "${title}" matches multiple step definitions. `
-                    : `Scenario "${title}" contains a step matching multiple step definitions. `) +
-                'Make the step definitions unique by removing duplicate or overlapping expressions.'
-            )
-            error = new Error(message.split('\n')[0])
-            error.stack = `${message}\n${this.getFeatureStack(uri, feature, scenario, step)}`
         }
 
-        // reporters show the reason of a skipped test from `pendingReason`, not from its error
-        const pendingReason = result.status === Status.AMBIGUOUS && state === 'skip' ? result.message : undefined
         const common = {
             title: title,
             state,
@@ -305,25 +272,6 @@ export default class CucumberFormatter extends Formatter {
         this.emit('test:' + state, payload)
     }
 
-    /**
-     * stack lines with the location of the scenario and the step in the feature file
-     */
-    private getFeatureStack(uri: string, feature: Feature, scenario: Pickle, step?: PickleStep) {
-        let stack = `\tat Feature(${uri}):1:1\n`
-        const children = feature.children?.flatMap((child) => child.rule ? child.rule.children : [child]) ?? []
-        const featScenario = children.find(c => scenario.astNodeIds && c.scenario?.id === scenario.astNodeIds[0])?.scenario
-        if (featScenario) {
-            stack += `\tat Scenario(${featScenario.name}):${featScenario.location?.line}:${featScenario.location?.column}\n`
-
-            const steps = [...children.flatMap((c) => c.background?.steps ?? []), ...(featScenario.steps ?? [])]
-            const featStep = steps.find(s => step?.astNodeIds && s.id === step.astNodeIds[0])
-            if (featStep) {
-                stack += `\tat Step(${featStep.text}):${featStep.location?.line}:${featStep.location?.column}\n`
-            }
-        }
-        return stack
-    }
-
     onGherkinDocument(gherkinDocEvent: GherkinDocument) {
         this.updateCurrentPickle({
             uri: gherkinDocEvent.uri,