Freegle / Iznik / 23559

72%

push

circleci

fix(test): stop overlapping substring assertions starving ripple:expand dry-run test

test_dry_run_is_exempt_from_the_single_instance_lock asserted both 'DRY RUN'
and 'Initialised:'. ripple:expand's summary line is prefixed '[DRY RUN]
Initialised: ...', so that one line contains BOTH substrings. Laravel's
PendingCommand registers one Mockery doWrite matcher per expected substring and
routes a matching line to the FIRST matcher only, so 'DRY RUN' swallowed the
summary line and the 'Initialised:' matcher never fired -> false
"Output does not contain 'Initialised:'" (master CI red, pipeline #8410).

Production output is correct and unchanged; assert the non-overlapping
'no reach will be written' notice line instead, which still proves the dry-run
body ran despite the held lock.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>

11301 of 15028 branches covered (75.2%)

Branch coverage included in aggregate %.

123909 of 174384 relevant lines covered (71.06%)

37.31 hits per line