fix: attempt different content type for windows

guidepup · Dec 28, 2023 · b05bc3c · b05bc3c
1 parent ebad7a9
commit b05bc3c
Show file tree

Hide file tree

Showing 3 changed files with 1 addition and 7 deletions.
diff --git a/src/attachRecording.ts b/src/attachRecording.ts
@@ -3,18 +3,15 @@ import { basename } from "path";
 import { test as playwrightTest } from "@playwright/test";
 import { delay } from "./delay";
 
-const MP4 = "video/mp4";
 const MOV = "video/quicktime";
 
 const EXISTS_RETRIES = 20;
 const EXISTS_WAIT = 100;
 
 export const attachRecording = async ({
-  osPlatform,
   path,
   test,
 }: {
-  osPlatform: NodeJS.Platform;
   path: string;
   test: typeof playwrightTest;
 }) => {
@@ -42,7 +39,6 @@ export const attachRecording = async ({
   }
 
   const name = basename(path);
-  const contentType = osPlatform === "darwin" ? MOV : MP4;
 
-  await test.info().attach(name, { contentType, path });
+  await test.info().attach(name, { contentType: MOV, path });
 };
diff --git a/src/macOsVoiceOver.spec.ts b/src/macOsVoiceOver.spec.ts
@@ -214,7 +214,6 @@ const generateTestSuite = ({
 
             try {
               await attachRecording({
-                osPlatform,
                 path: stopRecording?.(),
                 test,
               });

diff --git a/src/windowsNvda.spec.ts b/src/windowsNvda.spec.ts
@@ -185,7 +185,6 @@ const generateTestSuite = ({
 
             try {
               await attachRecording({
-                osPlatform,
                 path: stopRecording?.(),
                 test,
               });