reporter: remove color symbols from stdout / stderr

src/reporter/cucumber/messagesBuilder/AttachmentMapper.ts

@@ -59,6 +59,7 @@ import { AutofillMap } from '../../../utils/AutofillMap';
 import { collectStepsWithCategory, getHooksRootPwStep } from './pwStepUtils';
 import { PwAttachment } from '../../../playwright/types';
 import { isBddDataAttachment } from '../../../run/bddDataAttachment';
+import { stripAnsiEscapes } from '../../../utils/stripAnsiEscapes';
 
 export class AttachmentMapper {
   private stepAttachments = new AutofillMap<pw.TestStep, PwAttachment[]>();
@@ -115,7 +116,9 @@ export class AttachmentMapper {
   private mapStdoutAttachments(name: 'stdout' | 'stderr') {
     // map stdout / stderr to the 'After Hooks' step
     if (!this.result[name]?.length) return;
-    const body = this.result[name].join('');
+    const body = this.result[name]
+      .map((s) => (typeof s === 'string' ? stripAnsiEscapes(s) : s))
+      .join('');
     const afterHooksRoot = this.getAfterHooksRoot();
     const stepAttachments = this.stepAttachments.getOrCreate(afterHooksRoot, () => []);
     stepAttachments.push({

test/reporter-cucumber-html/check-report/passed.test.ts

@@ -49,7 +49,7 @@ test('Scenario: Scenario with attachments', async ({ page }) => {
     'text attachmentsome text', // no space between 'attachment' and 'some'
     'image attachment inline',
     'image attachment as file',
-    'stdoutsome logs',
+    'stdout123 some logs',
   ]);
 });
 

test/reporter-cucumber-html/features/steps.ts

@@ -29,7 +29,7 @@ Then('attach image as file', async ({ $testInfo }) => {
 });
 
 Then('attach stdout', async () => {
-  console.log('some logs'); // eslint-disable-line no-console
+  console.log(123, 'some logs'); // eslint-disable-line no-console
   // don't test console.error b/c it poisons the output
 });