Freegle / Iznik / 5314

68%

master: 70%

Build Type

Pull #228

circleci

Committed by edwh

Commit Message

fix(go): queue postcode remap when a location is excluded from a group

ExcludeLocation inserted into locations_excluded but never kicked off a
remap, so postcodes that had been mapped to the excluded area stayed
pointing at it. PostcodeRemapService already joins locations_excluded;
it just needs a fresh remap task for the affected polygon to recompute
KNN and move those postcodes to the next nearest non-excluded area.

Mirrors the UpdateLocation remap pattern: fetch ST_AsText of the
location's ourgeometry (falling back to geometry) and queue a
TaskRemapPostcodes with the polygon. For byname excludes, queue one
task per additional excluded location.

Test added: TestExcludeLocationQueuesRemapTask creates a polygon
location, POSTs Exclude, and asserts a remap_postcodes task is
enqueued in background_tasks for that location.

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

Pull Request Pull Request #228: fix(go): queue postcode remap when a location is excluded from a group

Coverage Stats

8865 of 10243 branches covered (86.55%)

Branch coverage included in aggregate %.

15 of 16 new or added lines in 1 file covered. (93.75%)

93 existing lines in 3 files now uncovered.

84399 of 127165 relevant lines covered (66.37%)

18.55 hits per line