fix: flakey tests due to cron-timeout misalignment

Cron waits for up to 120 seconds between runs.
The test has a 120 second timeout.
So if cron is at the end of its random wait when the tests start, there can be a timeout.
This is an edge case that causes flakiness.
We bump the timeout to 150s to provide a buffer.
These tests need to wait for a sync to happen, so they need to wait for the cron job to run.

Author: nams1570

apps/e2e/tests/backend/endpoints/api/v1/external-db-sync-advanced.test.ts

@@ -115,7 +115,7 @@ describe.sequential('External DB Sync - Advanced Tests', () => {
           throw err;
         }
       },
-      { description: 'User A to appear in both Project A databases', timeoutMs: 120000 }
+      { description: 'User A to appear in both Project A databases', timeoutMs: 150000 }
     );
 
     await waitForCondition(
@@ -130,7 +130,7 @@ describe.sequential('External DB Sync - Advanced Tests', () => {
           throw err;
         }
       },
-      { description: 'User B to appear in all three Project B databases', timeoutMs: 120000 }
+      { description: 'User B to appear in all three Project B databases', timeoutMs: 150000 }
     );
 
     const resA1 = await clientA1.query(`SELECT * FROM "users" WHERE "primary_email" = $1`, ['user-a@example.com']);
@@ -438,7 +438,7 @@ describe.sequential('External DB Sync - Advanced Tests', () => {
           const res = await externalClient.query(`SELECT COUNT(*) as count FROM "users"`);
           return parseInt(res.rows[0].count) >= userCount;
         },
-        { description: `all ${userCount} users to be synced`, timeoutMs: 120000 }
+        { description: `all ${userCount} users to be synced`, timeoutMs: 150000 }
       );
 
       const res = await externalClient.query(`SELECT COUNT(*) as count FROM "users"`);
@@ -500,7 +500,7 @@ describe.sequential('External DB Sync - Advanced Tests', () => {
           throw err;
         }
       },
-      { description: 'initial 3 users sync', timeoutMs: 120000 }
+      { description: 'initial 3 users sync', timeoutMs: 150000 }
     );
 
     let res = await client.query(`SELECT COUNT(*) as count FROM "users"`);
@@ -543,7 +543,7 @@ describe.sequential('External DB Sync - Advanced Tests', () => {
           throw err;
         }
       },
-      { description: 'final sync state correct', timeoutMs: 120000 }
+      { description: 'final sync state correct', timeoutMs: 150000 }
     );
 
     res = await client.query(`SELECT * FROM "users" ORDER BY "primary_email"`);
@@ -728,7 +728,7 @@ $$;`);
         );
         return res.rows.length === 0;
       },
-      { description: 'deleted user should never appear', timeoutMs: 120000 }
+      { description: 'deleted user should never appear', timeoutMs: 150000 }
     );
 
     const res = await client.query(
@@ -811,7 +811,7 @@ $$;`);
         );
         return followUp.rows.length === 1 && followUp.rows[0].display_name === 'Recreated Export';
       },
-      { description: 'recreated row persists after extra sync', timeoutMs: 120000 },
+      { description: 'recreated row persists after extra sync', timeoutMs: 150000 },
     );
   }, TEST_TIMEOUT);
 

apps/e2e/tests/backend/endpoints/api/v1/external-db-sync-race.test.ts

@@ -240,7 +240,7 @@ describe.sequential('External DB Sync - Race Condition Tests', () => {
           const res = await externalClient.query(`SELECT COUNT(*) AS count FROM "users"`);
           return parseInt(res.rows[0].count, 10) === totalUsers;
         },
-        { description: 'initial >300 users exported', timeoutMs: 120000 },
+        { description: 'initial >300 users exported', timeoutMs: 150000 },
       );
 
       // Delete user at index 1 (low sequence ID)
@@ -396,7 +396,7 @@ describe.sequential('External DB Sync - Race Condition Tests', () => {
             },
             {
               description: 'waiting for marker user to sync to external DB',
-              timeoutMs: 120_000,
+              timeoutMs: 150_000,
             },
           );
 

apps/e2e/tests/backend/endpoints/api/v1/external-db-sync-utils.ts

@@ -175,7 +175,7 @@ export async function waitForSyncedData(client: Client, email: string, expectedN
     },
     {
       description: `data for ${email} to appear in external DB`,
-      timeoutMs: 120000,
+      timeoutMs: 150000,
       intervalMs: 500,
     }
   );
@@ -200,7 +200,7 @@ export async function waitForSyncedDeletion(client: Client, email: string) {
     },
     {
       description: `data for ${email} to be removed from external DB`,
-      timeoutMs: 120000,
+      timeoutMs: 150000,
       intervalMs: 500,
     }
   );
@@ -224,7 +224,7 @@ export async function waitForTable(client: Client, tableName: string) {
     },
     {
       description: `table ${tableName} to be created`,
-      timeoutMs: 120000,
+      timeoutMs: 150000,
       intervalMs: 500,
     }
   );