From 4c2d62a8816bb606e1e73b1c82fa39aa13406a70 Mon Sep 17 00:00:00 2001 From: Remigiusz Dudek Date: Mon, 14 Oct 2024 14:16:19 +0200 Subject: [PATCH] feat(html-reporter): recognize video attachment by the contentType (#33074) Closes https://github.com/microsoft/playwright/issues/33073. --------- Signed-off-by: Remigiusz Dudek Co-authored-by: Dmitry Gozman --- packages/html-reporter/src/testResultView.tsx | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/packages/html-reporter/src/testResultView.tsx b/packages/html-reporter/src/testResultView.tsx index 273703a0c7064..48a24a2391f27 100644 --- a/packages/html-reporter/src/testResultView.tsx +++ b/packages/html-reporter/src/testResultView.tsx @@ -70,7 +70,7 @@ export const TestResultView: React.FC<{ const { screenshots, videos, traces, otherAttachments, diffs, errors, htmls } = React.useMemo(() => { const attachments = result?.attachments || []; const screenshots = new Set(attachments.filter(a => a.contentType.startsWith('image/'))); - const videos = attachments.filter(a => a.name === 'video'); + const videos = attachments.filter(a => a.contentType.startsWith('video/')); const traces = attachments.filter(a => a.name === 'trace'); const htmls = attachments.filter(a => a.contentType.startsWith('text/html')); const otherAttachments = new Set(attachments);