test: cover user-provider reconnect-resync branch #106

New Issue

Ghost · 2026-06-21T05:05:39+03:00

Ghost commented

2026-06-21 05:05:39 +03:00

PR #101 (commit e52f069f, #66) added reconnect-resync to apps/client/src/features/user/user-provider.tsx: a firstConnect flag so that only on a socket reconnect it invalidates the root-sidebar-pages and sidebar-pages query keys (first connect must NOT invalidate).

Gap: untested; the project has no socket/provider test harness yet.

Needed: if a fake-socket/provider harness is feasible (Vitest, like the provider tests added in this PR), mount UserProvider with a stubbed socket + queryClient, emit connect twice, and assert invalidateQueries fires only on the second connect, for both keys.

Found in code review of PR #101.

PR #101 (commit e52f069f, #66) added reconnect-resync to apps/client/src/features/user/user-provider.tsx: a `firstConnect` flag so that only on a socket **reconnect** it invalidates the `root-sidebar-pages` and `sidebar-pages` query keys (first connect must NOT invalidate). **Gap:** untested; the project has no socket/provider test harness yet. **Needed:** if a fake-socket/provider harness is feasible (Vitest, like the provider tests added in this PR), mount `UserProvider` with a stubbed socket + queryClient, emit `connect` twice, and assert `invalidateQueries` fires only on the second connect, for both keys. _Found in code review of PR #101._

Ghost added the test label 2026-06-21 05:05:39 +03:00

vvzvlad referenced this issue from a commit

2026-06-21 05:24:26 +03:00

fix(review): address PR #101 review findings (dead DI, docs)

Ghost referenced this issue from a commit

2026-06-21 05:49:36 +03:00

test(ws): cover the user-provider reconnect-resync branch (#106)

Ghost referenced this issue

2026-06-21 05:51:20 +03:00

fix: review/red-team batch 2 — 30 issues (security, ws, page-templates, html-embed, mcp, tests, docs) #101

Ghost referenced this issue

2026-06-21 05:53:55 +03:00