From 5778201462e25372addc1860fc7789a6b8bbee15 Mon Sep 17 00:00:00 2001 From: Remigiusz Dudek Date: Mon, 14 Oct 2024 13:08:23 +0200 Subject: [PATCH] Update packages/html-reporter/src/testResultView.tsx fair point, I wanted to be conservative in order not to introduce regression but if you say that all video types are supported I'm ok with your suggestion. Co-authored-by: Dmitry Gozman Signed-off-by: Remigiusz Dudek --- 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 0bc1235638..48a24a2391 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.contentType === 'video/webm'); + 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);