Handle UNKNOWN status when rendering test run outcome (#2441)

* added unknown status for summary message * spec total count updated * match order from messages * changelog and attribution --------- Co-authored-by: David Goss <[email protected]>
cucumber · Nov 16, 2024 · ee538bd · ee538bd
1 parent 1c4d9c5
commit ee538bd
Show file tree

Hide file tree

Showing 4 changed files with 8 additions and 1 deletion.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -14,6 +14,9 @@ Please see [CONTRIBUTING.md](./CONTRIBUTING.md) on how to contribute to Cucumber
 ### Changed
 - Replace JUnit formatter with messages-based package ([#2445](https://github.com/cucumber/cucumber-js/pull/2445))
 
+### Fixed
+- Handle UNKNOWN status when rendering test run outcome ([#2441](https://github.com/cucumber/cucumber-js/pull/2441))
+
 ## [11.0.1] - 2024-09-14
 ### Fixed
 - Add missing setParallelCanAssign export ([#2427](https://github.com/cucumber/cucumber-js/pull/2427))

diff --git a/package.json b/package.json
@@ -157,6 +157,7 @@
     "Simon Lampen <[email protected]>",
     "Sonny Piers <[email protected]>",
     "Stanley Shyiko <[email protected]>",
+    "Steve Hynding (https://github.com/hynding)",
     "Steve Tooke <[email protected]>",
     "szymonprz <[email protected]>",
     "Ted de Koning",

diff --git a/src/formatter/helpers/summary_helpers.ts b/src/formatter/helpers/summary_helpers.ts
@@ -11,6 +11,7 @@ const STATUS_REPORT_ORDER = [
   messages.TestStepResultStatus.PENDING,
   messages.TestStepResultStatus.SKIPPED,
   messages.TestStepResultStatus.PASSED,
+  messages.TestStepResultStatus.UNKNOWN,
 ]
 
 export interface IFormatSummaryRequest {

diff --git a/src/formatter/helpers/summary_helpers_spec.ts b/src/formatter/helpers/summary_helpers_spec.ts
@@ -217,14 +217,16 @@ describe('SummaryHelpers', () => {
           '    Given a skipped step',
           '  Scenario: a6',
           '    Given an undefined step',
+          // an unknown scenario
+          '  Scenario:',
         ].join('\n')
 
         // Act
         const output = await testFormatSummary({ sourceData })
 
         // Assert
         expect(output).to.contain(
-          '6 scenarios (1 failed, 1 ambiguous, 1 undefined, 1 pending, 1 skipped, 1 passed)\n' +
+          '7 scenarios (1 failed, 1 ambiguous, 1 undefined, 1 pending, 1 skipped, 1 passed, 1 unknown)\n' +
             '6 steps (1 failed, 1 ambiguous, 1 undefined, 1 pending, 1 skipped, 1 passed)\n' +
             '0m00.000s (executing steps: 0m00.000s)\n'
         )