Freegle / Iznik / 5673

72%

master: 70%

Pull #244

circleci

fix(tests): fix PR #234 CI failures — Playwright wrong groupId + Coveralls vitest coverage

Root causes and fixes:

1. Playwright test-modtools-edits-flow (timeout):
   postMessage() picks a group from the postcode area, not necessarily
   testEnv.group.id. Using testEnv.group.id in the Approve API call
   silently updated 0 rows (wrong groupid), leaving collection='Pending',
   which prevented edit review creation and caused the 202.5s timeout.
   Fix: fetch the message first to get actualGroupId, use that in both
   the Approve and PATCH /memberships calls.

2. Playwright test-repost-group-change (wrong group ID 69615 vs 55):
   Used .first() to find rejected message cards, picking stale messages
   from previous CI runs that belonged to group 69615. Fix: target the
   specific rejected message by data-message-id attribute so only the
   message created by this test run is found.

3. Coveralls vitest coverage decrease:
   auth.js disableGoogleAutoselect had 5 uncovered lines (window guard,
   happy-path disableAutoSelect, catch block) and a bare `window`
   reference that throws ReferenceError in SSR / post-teardown contexts.
   Fix: rewrite guard as typeof window === 'undefined' check and add
   3-test describe block covering undefined-window, happy-path, and
   error-catch branches.

Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com>

13412 of 20348 branches covered (65.91%)

Branch coverage included in aggregate %.

96292 of 132848 relevant lines covered (72.48%)

21.5 hits per line