Freegle / Iznik / 8646

70%

Build Type

push

circleci

Committed by edwh

Commit Message

fix(playwright): two intermittent CI failures on master (#8634)

ComposeGroup.vue: add pre-selected group as fallback option
When a repost flow pre-selects group 55 but authStore.groups isn't
populated yet (fetchUser() is async), groupOptions only contains
groupsnear entries. b-form-select resets its display to the first
available option (e.g. 69615), and Playwright reads the wrong value.
Fix: append current composeStore.group to groupOptions using
groupStore.get() for the label, so the option always exists at render.

test-modtools-edits-flow: poll until message groups populated
Immediately after postMessage(), GET /message/{id} may return
groups:[] if the server hasn't finished processing. Using the fallback
testEnv.group.id (55) in the Approve call silently updates 0 rows
(message is in a different group), leaving collection=Pending so no
edit review is created and the modtools edits page shows nothing.
Fix: poll with expect.poll() until groups.length > 0 before approving.

ComposeGroup.spec.js: add useGroupStore mock + two new test cases
Without a mock for ~/stores/group, the new useGroupStore() call in
the component would throw "No active Pinia" in Vitest. Added mock
with get() → null, plus tests for the new fallback-option behaviour.

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

Coverage Stats

13636 of 20642 branches covered (66.06%)

Branch coverage included in aggregate %.

14 of 14 new or added lines in 1 file covered. (100.0%)

29 existing lines in 4 files now uncovered.

98354 of 134248 relevant lines covered (73.26%)

22.31 hits per line