fix: Add test user to backend test setup to satisfy foreign key const…

[marcodejongh]

…raint

The session-persistence tests use 'user-123' as the userId for createDiscoverableSession, but the board_sessions.created_by_user_id column has a foreign key constraint referencing the users table.

This fix:

• Truncates the users table in beforeEach to ensure clean state
• Creates the 'user-123' test user before each test

[vercel]

The latest updates on your projects. Learn more about Vercel for GitHub.

Project	Deployment	Review	Updated (UTC)
boardsesh	Ready	Preview, Comment	Dec 31, 2025 10:26am

[claude]

Claude Review

No issues found.

The fix correctly addresses the foreign key constraint issue by:

1. Truncating the users table in the proper order (after dependent tables)
2. Creating the required test user before tests that reference it

The ON CONFLICT DO NOTHING is appropriate for test setup idempotency.

[claude]

Claude Review

No significant issues found.

The fix correctly addresses the foreign key constraint violation by:

1. Truncating the users table in beforeEach
2. Creating the 'user-123' test user before tests run
3. Using sharedDb for consistency with roomManager

Minor note: The ON CONFLICT (id) DO NOTHING clause is unnecessary since the table is truncated immediately before, but it's harmless.

[claude]

Claude Review

Issues Found

1. Schema drift in test table definitions (packages/backend/src/__tests__/setup.ts:75-155): The kilter_climbs and tension_climbs test tables are missing columns present in the actual schema:

   • Missing angle column
   • Missing synced column

   This may cause tests to pass against an incomplete schema while failing against the real database.

2. Weak test assertions (packages/backend/src/__tests__/climb-queries.test.ts): Several tests have assertions that always pass:

   • Line 73-78: The expect(climb).toBeDefined() inside the grade filter test doesn't actually validate the grade range
   • Line 264-265: expect(kilterResult === null || typeof kilterResult === 'object').toBe(true) - this will always be true since the result is either null or an object

3. Tests don't verify core functionality (packages/backend/src/__tests__/climb-queries.test.ts): The climb query tests rely on external database state (synced production-like data) rather than setting up known test fixtures. If the database is empty, tests like minAscents filter (line 81-95) will pass vacuously with 0 results.