From a97a7c174ec840133ed981a19bbed8702483a3e7 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Hanno=20J=2E=20G=C3=B6decke?= Date: Fri, 6 Dec 2024 16:40:00 +0100 Subject: [PATCH] Update src/libs/E2E/utils/NetworkInterceptor.ts Co-authored-by: Tim Golen --- src/libs/E2E/utils/NetworkInterceptor.ts | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/libs/E2E/utils/NetworkInterceptor.ts b/src/libs/E2E/utils/NetworkInterceptor.ts index 0e453e250f77..8a29925e288e 100644 --- a/src/libs/E2E/utils/NetworkInterceptor.ts +++ b/src/libs/E2E/utils/NetworkInterceptor.ts @@ -156,7 +156,7 @@ export default function installNetworkInterceptor( const headers = getFetchRequestHeadersAsObject(options); const url = fetchArgsGetUrl(args); - // Don't process these specific API commands because running them over and over again in the tests hammers the server in a bad way. + // Don't process these specific API commands because running them over and over again in the tests increases the size of the notificationPreferences NVP on the server to an infinite size. This is due to the NVP storing this setting once for each user's device, and since the E2E tests use AWS device farm, the user ends up with thousands of different devices, unlike normal users that might only ever have about a dozen. We found the NVP was over 2.5mb in size and that slows down database replication. if (url.includes('OptInToPushNotifications') || url.includes('OptOutOfPushNotifications')) { console.debug('Skipping request to opt in or out of push notifications'); return Promise.resolve(new Response());