Freegle / Iznik / 11347

73%

Build Type

push

circleci

Committed by edwh

Commit Message

fix(batch): update dry-run tests to match new pass-through architecture

Commit 132a6bad7 changed dry-run commands to call services with dryRun=true
rather than early-returning. Tests still expected never() on service methods.

- Update 16 dry-run command tests to expect once() with dryRun=true arg
- Fix MessageExpiryService::processExpiredFromSpatialIndex to process all
  spatial entries (not only those with pre-existing EXPIRED outcomes), creating
  EXPIRED outcome only when none exists — matches test specifications
- Fix UpdateSpatialIndexCommandTest non-dry-run test: willReturn(array) not int

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

Coverage Stats

13803 of 20749 branches covered (66.52%)

Branch coverage included in aggregate %.

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

1 existing line in 1 file now uncovered.

101759 of 137857 relevant lines covered (73.81%)

22.46 hits per line