Fix snapshot timestamp collision in tests

Add delays between branch creates that share the same parent to prevent ZFS snapshot name collisions. Snapshots use second-precision timestamps, so rapid branch creation from the same parent causes duplicate snapshot names.

Also add TIMEOUTS.PITR_RECOVERY constant for semantic clarity on PITR test timeouts.

Fixes:
- branch-delete-force.test.ts: Add sleeps between sibling branches from same parent
- helpers/wait.ts: Add TIMEOUTS.PITR_RECOVERY (180s) constant
- pitr.test.ts: Use TIMEOUTS.PITR_RECOVERY instead of hardcoded 180000

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>

Author: elitan

tests/branch-delete-force.test.ts

@@ -135,6 +135,9 @@ describe('Branch Delete with --force', () => {
       await branchCreateCommand('force-test/level2a', { parent: 'force-test/level1' });
       await waitForBranchReady('force-test', 'level2a');
 
+      // Small delay to ensure unique snapshot timestamps (snapshots use second-precision)
+      await Bun.sleep(1000);
+
       // Create level2b (child of level1)
       await branchCreateCommand('force-test/level2b', { parent: 'force-test/level1' });
       await waitForBranchReady('force-test', 'level2b');
@@ -174,7 +177,7 @@ describe('Branch Delete with --force', () => {
       // Only main should remain
       expect(project?.branches?.length).toBe(1);
       expect(project?.branches?.[0]?.name).toBe('force-test/main');
-    }, { timeout: 60000 });
+    }, { timeout: 60000 }); // Creates 4 branches, needs extra time
   });
 
   describe('Delete branch with sibling branches', () => {
@@ -188,6 +191,9 @@ describe('Branch Delete with --force', () => {
       await branchCreateCommand('force-test/sibling1', {});
       await waitForBranchReady('force-test', 'sibling1');
 
+      // Small delay to ensure unique snapshot timestamps (snapshots use second-precision)
+      await Bun.sleep(1000);
+
       await branchCreateCommand('force-test/sibling2', {});
       await waitForBranchReady('force-test', 'sibling2');
 
@@ -215,6 +221,6 @@ describe('Branch Delete with --force', () => {
       expect(project?.branches?.find((b: any) => b.name === 'force-test/sibling1')).toBeUndefined();
       expect(project?.branches?.find((b: any) => b.name === 'force-test/sibling1-child')).toBeUndefined();
       expect(project?.branches?.find((b: any) => b.name === 'force-test/sibling2')).toBeDefined();
-    }, { timeout: 60000 });
+    }, { timeout: 60000 }); // Creates 3 branches, needs extra time
   });
 });

tests/helpers/wait.ts

@@ -15,6 +15,7 @@ export const TIMEOUTS = {
   CONTAINER_STOP: 10_000,
   DATASET_CREATE: 5_000,
   POSTGRES_READY: 60_000,
+  PITR_RECOVERY: 180_000, // PITR recovery needs extra time for WAL replay
 } as const;
 
 /**

tests/integration.test.ts

@@ -60,7 +60,7 @@ describe('Integration Tests', () => {
       // Branches should be much smaller due to CoW
       expect(devSize).toBeLessThan(parentSize);
       expect(stagingSize).toBeLessThan(parentSize);
-    }, { timeout: 60000 });
+    }, { timeout: 60000 }); // Inserts 100 rows, needs extra time
   });
 
   describe('State Integrity', () => {
@@ -139,6 +139,6 @@ describe('Integration Tests', () => {
       const project = state.projects?.find((p: any) => p.name === 'multi-test');
 
       expect(project?.branches?.length).toBe(3); // main, dev, feature
-    }, { timeout: 60000 });
+    }, { timeout: 60000 }); // Creates multiple branches, needs extra time
   });
 });

tests/pitr.test.ts

@@ -6,6 +6,7 @@
 import { describe, test, expect, beforeAll, afterAll } from 'bun:test';
 import * as cleanup from './helpers/cleanup';
 import { getProjectCredentials, getBranchPort, query, waitForReady, ensureWALArchived } from './helpers/database';
+import { TIMEOUTS } from './helpers/wait';
 import {
   silenceConsole,
   projectCreateCommand,
@@ -87,7 +88,7 @@ describe('Point-in-Time Recovery (PITR)', () => {
       // Should NOT have: 'after-snapshot-2' (inserted after recovery target)
       const count = await query(recoveryPort, creds.password, 'SELECT COUNT(*) FROM pitr_data;');
       expect(parseInt(count)).toBe(2);
-    }, { timeout: 180000 }); // PITR recovery needs extra time for WAL replay
+    }, { timeout: TIMEOUTS.PITR_RECOVERY });
 
     test('should fail with PITR before any snapshots', async () => {
       // Try to recover to a time way in the past
@@ -107,6 +108,6 @@ describe('Point-in-Time Recovery (PITR)', () => {
         // Either success or failure is acceptable for this test
         expect(error).toBeDefined();
       }
-    }, { timeout: 180000 }); // PITR recovery needs extra time for WAL replay on slower machines
+    }, { timeout: TIMEOUTS.PITR_RECOVERY });
   });
 });

tests/wal.test.ts

@@ -38,7 +38,7 @@ describe('WAL Operations', () => {
     test('should show WAL info for all branches', async () => {
       await ensureSetup();
       await walInfoCommand(undefined);
-    }, { timeout: 60000 }); // PostgreSQL startup + WAL archiving can take time
+    }, { timeout: 60000 });
 
     test('should show WAL info for specific branch', async () => {
       await ensureSetup();