Freegle / Iznik / 9566

72%

master: 70%

Pull #354

circleci

fix(tests): prevent cross-run test data pollution in Playwright tests

Two tests were failing in CI due to non-idempotent test state:

1. test-repost-group-change: The Go API groupid-persistence fix (PR #349)
   writes the user-selected group to messages_drafts on PATCH /message. A
   prior test run that changed the group dropdown left the wrong groupid in
   the draft, causing the next run's repost form to pre-select the stale
   group. Fix: create-test-env.php now resets messages_drafts.groupid to
   the test group id on every env refresh.

2. test-modtools-edits-flow: postMessage() uses the testEnv postcode
   (WR1 2ET for mteditsflow). Real Freegle groups covering that postcode
   appear first in groupsnear, so the message was posted to a real group
   the test mod cannot moderate. Fix: add a groupId option to the
   postMessage fixture that explicitly selects the specified group in the
   whereami dropdown; pass groupId: testEnv.group.id in the edits flow test.

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

13595 of 20538 branches covered (66.19%)

Branch coverage included in aggregate %.

98330 of 134464 relevant lines covered (73.13%)

22.44 hits per line