From c58bfd0552da433cf69cc3ed8eadd1041e75a364 Mon Sep 17 00:00:00 2001 From: Ross Wollman Date: Sat, 3 Sep 2022 18:25:56 -0400 Subject: [PATCH] fix(html-reporter): duplicate attachment name (#16929) Fixes #16456. The extra span/name was coming from https://github.com/microsoft/playwright/blob/077b8a928960493a4cf275bc4fc4d4176b2a6098/packages/html-reporter/src/links.tsx#L72 (in conjunction with the line above it). These two lines assume `path` and `body` are mutally exclusive. Regression likely introduced by the PR that added the now removed line #10778. --- packages/playwright-test/src/reporters/html.ts | 1 - 1 file changed, 1 deletion(-) diff --git a/packages/playwright-test/src/reporters/html.ts b/packages/playwright-test/src/reporters/html.ts index 8ef1b4e3e0..4fd9c75a14 100644 --- a/packages/playwright-test/src/reporters/html.ts +++ b/packages/playwright-test/src/reporters/html.ts @@ -503,7 +503,6 @@ class HtmlBuilder { name: a.name, contentType: a.contentType, path: 'data/' + sha1, - body: a.body, }; }