fix(tradein): чинит main-red — патч _geoportal_house_match в cadastral-тесте (хвост #1786) #1787

Merged

lekss361 merged 1 commit from fix/tradein-geoportal-test-cadastral into main

2026-06-19 15:07:32 +00:00

lekss361 commented

2026-06-19 15:06:47 +00:00

Owner

Проблема

PR #1786 (ингест геопортала ЕКБ) смержили на коммите 64d113d — до того как тест-фикс попал в ветку. В результате main красный: tests/services/test_cadastral_reverse.py::test_geocode_uses_cadastral_before_yandex падает.

Причина: новый geoportal-tier _geoportal_house_match() стоит первым в geocode(), но этот legacy-тест его не мокал → MagicMock-db протекает через новый tier (lat=1.0) вместо ожидаемого cadastral-пути. Воркер пропатчил аналогичные тесты в test_geocoder_cadastral_matcher.py, но этот в другом файле пропустил, а merge опередил мой post-fix push в ветку #1786.

Фикс

Один патч в with-блок теста:

patch("app.services.geocoder._geoportal_house_match", return_value=None),

→ новый tier мимо → тест проверяет cadastral-путь как задумано.

Локально: pytest ...::test_geocode_uses_cadastral_before_yandex → 1 passed. Это единственный geocode()-тест в файле (проверил grep), CI и показывал ровно 1 фейл. Pre-commit/ruff green.

## Проблема PR #1786 (ингест геопортала ЕКБ) смержили на коммите `64d113d` — **до** того как тест-фикс попал в ветку. В результате **main красный**: `tests/services/test_cadastral_reverse.py::test_geocode_uses_cadastral_before_yandex` падает. Причина: новый geoportal-tier `_geoportal_house_match()` стоит **первым** в `geocode()`, но этот legacy-тест его не мокал → MagicMock-db протекает через новый tier (`lat=1.0`) вместо ожидаемого cadastral-пути. Воркер пропатчил аналогичные тесты в `test_geocoder_cadastral_matcher.py`, но этот в другом файле пропустил, а merge опередил мой post-fix push в ветку #1786. ## Фикс Один патч в `with`-блок теста: ```python patch("app.services.geocoder._geoportal_house_match", return_value=None), ``` → новый tier мимо → тест проверяет cadastral-путь как задумано. Локально: `pytest ...::test_geocode_uses_cadastral_before_yandex` → **1 passed**. Это единственный `geocode()`-тест в файле (проверил grep), CI и показывал ровно 1 фейл. Pre-commit/ruff green.

lekss361 added 1 commit 2026-06-19 15:06:48 +00:00

test(tradein): patch _geoportal_house_match in cadastral-before-yandex test

CI / changes (pull_request) Successful in 6s

Details

CI / backend-tests (pull_request) Has been skipped

Details

CI / frontend-tests (pull_request) Has been skipped

Details

CI / openapi-codegen-check (pull_request) Has been skipped

Details

91b04a1851

#1786 merged before the test fix landed → main red on
test_geocode_uses_cadastral_before_yandex. The new geoportal house-match
tier runs first in geocode(); this legacy cadastral test must mock it to
None so it exercises the cadastral forward path (else the MagicMock db row
leaks through the new tier as lat=1.0).