@msw/playwright 0.6.3 → 0.6.4

package/build/index.js

@@ -37,7 +37,7 @@ var SetupPlaywrightApi = class extends SetupApi {
 			* requests through the matching logic below.
 			* @see https://github.com/mswjs/playwright/issues/13
 			*/
-			if (this.options.skipAssetRequests && isCommonAssetRequest(fetchRequest)) return route.fallback();
+			if (this.options.skipAssetRequests && isCommonAssetRequest(fetchRequest)) return this.safelyHandleRoute(() => route.fallback());
 			const handlers = this.handlersController.currentHandlers().filter((handler) => {
 				return handler instanceof RequestHandler;
 			});
@@ -51,14 +51,16 @@ var SetupPlaywrightApi = class extends SetupApi {
 				baseUrl
 			} });
 			if (response) {
-				if (response.status === 0) return route.abort();
-				return route.fulfill({
-					status: response.status,
-					headers: Object.fromEntries(response.headers),
-					body: response.body ? Buffer.from(await response.arrayBuffer()) : void 0
+				if (response.status === 0) return this.safelyHandleRoute(() => route.abort());
+				return this.safelyHandleRoute(async () => {
+					return route.fulfill({
+						status: response.status,
+						headers: Object.fromEntries(response.headers),
+						body: response.body ? Buffer.from(await response.arrayBuffer()) : void 0
+					});
 				});
 			}
-			return route.fallback();
+			return this.safelyHandleRoute(() => route.fallback());
 		});
 		await context.routeWebSocket(INTERNAL_MATCH_ALL_REG_EXP, async (route) => {
 			const allWebSocketHandlers = this.handlersController.currentHandlers().filter((handler) => {
@@ -90,6 +92,21 @@ var SetupPlaywrightApi = class extends SetupApi {
 		if (url === "about:blank") return;
 		return decodeURI(new URL(encodeURI(url)).origin);
 	}
+	async safelyHandleRoute(callback) {
+		try {
+			await callback();
+		} catch (error) {
+			/**
+			* @note Ignore "Route is already handled!" errors.
+			* Playwright has a bug where requests terminated due to navigation
+			* cause your in-flight route handlers to throw. There's no means to
+			* detect that scenario as both "route.handled" and "route._handlingPromise" are internal.
+			* @see https://github.com/mswjs/playwright/issues/35
+			*/
+			if (error instanceof Error && /route is already handled/i.test(error.message)) return;
+			throw error;
+		}
+	}
 };
 var PlaywrightWebSocketClientConnection = class {
 	id;

package/package.json

@@ -1,7 +1,7 @@
 {
   "type": "module",
   "name": "@msw/playwright",
-  "version": "0.6.3",
+  "version": "0.6.4",
   "description": "Mock Service Worker binding for Playwright",
   "main": "./build/index.js",
   "types": "./build/index.d.ts",

package/src/fixture.ts

@@ -99,7 +99,7 @@ class SetupPlaywrightApi extends SetupApi<LifeCycleEventsMap> {
           this.options.skipAssetRequests &&
           isCommonAssetRequest(fetchRequest)
         ) {
-          return route.fallback()
+          return this.safelyHandleRoute(() => route.fallback())
         }
 
         const handlers = this.handlersController
@@ -134,19 +134,21 @@ class SetupPlaywrightApi extends SetupApi<LifeCycleEventsMap> {
 
         if (response) {
           if (response.status === 0) {
-            return route.abort()
+            return this.safelyHandleRoute(() => route.abort())
           }
 
-          return route.fulfill({
-            status: response.status,
-            headers: Object.fromEntries(response.headers),
-            body: response.body
-              ? Buffer.from(await response.arrayBuffer())
-              : undefined,
+          return this.safelyHandleRoute(async () => {
+            return route.fulfill({
+              status: response.status,
+              headers: Object.fromEntries(response.headers),
+              body: response.body
+                ? Buffer.from(await response.arrayBuffer())
+                : undefined,
+            })
           })
         }
 
-        return route.fallback()
+        return this.safelyHandleRoute(() => route.fallback())
       },
     )
 
@@ -201,6 +203,30 @@ class SetupPlaywrightApi extends SetupApi<LifeCycleEventsMap> {
     // Encode/decode to preserve escape characters.
     return decodeURI(new URL(encodeURI(url)).origin)
   }
+
+  private async safelyHandleRoute(
+    callback: () => Promise<void>,
+  ): Promise<void> {
+    try {
+      await callback()
+    } catch (error) {
+      /**
+       * @note Ignore "Route is already handled!" errors.
+       * Playwright has a bug where requests terminated due to navigation
+       * cause your in-flight route handlers to throw. There's no means to
+       * detect that scenario as both "route.handled" and "route._handlingPromise" are internal.
+       * @see https://github.com/mswjs/playwright/issues/35
+       */
+      if (
+        error instanceof Error &&
+        /route is already handled/i.test(error.message)
+      ) {
+        return
+      }
+
+      throw error
+    }
+  }
 }
 
 class PlaywrightWebSocketClientConnection implements WebSocketClientConnectionProtocol {