comments

src/reporter/cucumber/messagesBuilder/TestRun.ts

@@ -17,6 +17,7 @@ export class TestRun {
   // See: https://github.com/cucumber/messages/blob/main/messages.md#testrunfinished
   // todo: there are also new props: message and exception,
   // they could be populated in reporter.onError handler
+  // See: https://github.com/cucumber/html-formatter/issues/340
   buildTestRunFinished({ status, startTime, duration }: pw.FullResult) {
     const testRunFinished: messages.TestRunFinished = {
       testRunStartedId: this.id,

test/reporter-cucumber-html/check-report/specs/error-in-after-all.test.ts

@@ -7,6 +7,7 @@ import { test } from '../fixtures';
 // It would be better to run afterAll hooks inside test.afterAll(),
 // but it's will work incorrectly if single afterAll hook is tagged to several features,
 // because it will be run several times.
+// Or just un-tagged afterAll hook + several files.
 
 test.describe.skip('error in anonymous after all hook', () => {
   test.use({ featureUri: 'error-in-after-all/anonymous.feature' });