Skip calling specific APIs in the E2E tests

Expensify · Dec 6, 2024 · 00d8bf0 · 00d8bf0
1 parent 73942b5
commit 00d8bf0
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 6 deletions.
diff --git a/src/libs/E2E/client.ts b/src/libs/E2E/client.ts
@@ -24,12 +24,6 @@ const defaultRequestInit: RequestInit = {
 };
 
 const sendRequest = (url: string, data: Record<string, unknown>): Promise<Response> => {
-    // Don't process these specific API commands because running them over and over again in the tests hammers the server in a bad way.
-    if (url.includes('command=OptInToPushNotifications') || url.includes('command=OptOutOfPushNotifications')) {
-        console.debug('Skipping request to opt in or out of push notifications');
-        return Promise.resolve(new Response());
-    }
-
     return fetch(url, {
         method: 'POST',
         headers: {

diff --git a/src/libs/E2E/utils/NetworkInterceptor.ts b/src/libs/E2E/utils/NetworkInterceptor.ts
@@ -155,6 +155,13 @@ export default function installNetworkInterceptor(
         const options = fetchArgsGetRequestInit(args);
         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.
+        if (url.includes('command=OptInToPushNotifications') || url.includes('command=OptOutOfPushNotifications')) {
+            console.debug('Skipping request to opt in or out of push notifications');
+            return Promise.resolve(new Response());
+        }
+
         // Check if headers contain any of the ignored headers, or if react native metro server:
         if (IGNORE_REQUEST_HEADERS.some((header) => headers[header] != null) || url.includes('8081')) {
             return originalFetch(...args);