fix: provider video download failure should not stop tests (#233)

* fix: device provider video fix

* chore: added changeset

* chore: updated changeset

.changeset/fuzzy-cherries-beg.md

@@ -0,0 +1,5 @@
+---
+"appwright": patch
+---
+
+fix: provider video download failure should not stop tests

src/providers/browserstack/index.ts

@@ -206,13 +206,15 @@ export class BrowserStackDeviceProvider implements DeviceProvider {
           });
           if (response.status !== 200) {
             // Retry if not 200
-            throw new Error(
+            logger.error(
               `Video not found: ${response.status} (URL: ${videoURL})`,
             );
+            return;
           }
           const reader = response.body?.getReader();
           if (!reader) {
-            throw new Error("Failed to get reader from response body.");
+            logger.error("Failed to get reader from response body.");
+            return;
           }
           const streamToFile = async () => {
             // eslint-disable-next-line no-constant-condition

src/providers/lambdatest/index.ts

@@ -189,13 +189,15 @@ export class LambdaTestDeviceProvider implements DeviceProvider {
           });
           if (response.status !== 200) {
             // Retry if not 200
-            throw new Error(
+            logger.error(
               `Video not found: ${response.status} (URL: ${videoURL})`,
             );
+            return;
           }
           const reader = response.body?.getReader();
           if (!reader) {
-            throw new Error("Failed to get reader from response body.");
+            logger.error("Failed to get reader from response body.");
+            return;
           }
           const streamToFile = async () => {
             // eslint-disable-next-line no-constant-condition