fix(msk): тест полос по округам разъехался с main — у _fetch_deals нет rooms
All checks were successful
CI / changes (pull_request) Successful in 10s
CI Trade-In / browser-tests (pull_request) Has been skipped
CI Trade-In / frontend-checks (pull_request) Has been skipped
CI Trade-In / changes (pull_request) Successful in 9s
CI / backend-tests (pull_request) Has been skipped
CI / frontend-tests (pull_request) Has been skipped
CI / openapi-codegen-check (pull_request) Has been skipped
CI Trade-In / backend-tests (pull_request) Successful in 5m7s
All checks were successful
CI / changes (pull_request) Successful in 10s
CI Trade-In / browser-tests (pull_request) Has been skipped
CI Trade-In / frontend-checks (pull_request) Has been skipped
CI Trade-In / changes (pull_request) Successful in 9s
CI / backend-tests (pull_request) Has been skipped
CI / frontend-tests (pull_request) Has been skipped
CI / openapi-codegen-check (pull_request) Has been skipped
CI Trade-In / backend-tests (pull_request) Successful in 5m7s
Post-merge прогон main после #3440 дал 5 failed: `_fetch_deals() got an unexpected keyword argument 'rooms'`. Семантический конфликт мержа, а не поломка: ветка отводилась до #3256, который УБРАЛ параметр `rooms` (сделки Росреестра комнатность не несут). Текстового конфликта git не увидел, pre-merge CI ветки был зелёным, красным стало только на объединённом дереве. В проде эффекта нет: `_fetch_deals` из `app/` не вызывается ни разу ни до, ни после мержа — функцию держат только эти тесты. `uv run python -m pytest tests/test_3051_moscow_okrug_bands.py` — 24 passed, ruff чист. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01VQ8jqr4SFirX5tFLwdSrXh
This commit is contained in:
parent
a99a9b870c
commit
37494dd74a
1 changed files with 5 additions and 1 deletions
|
|
@ -283,11 +283,15 @@ def _fetch(
|
|||
) -> list[dict[str, Any]]:
|
||||
est._city_price_bands_cache = None
|
||||
try:
|
||||
# `rooms` у `_fetch_deals` больше нет: #3256 убрал параметр, а не просто
|
||||
# перестал класть его в SQL — сделки Росреестра комнатность не несут.
|
||||
# Тест писался на ветке, отведённой до того тикета, и разъехался с main
|
||||
# молча: текстового конфликта в мерже не было, красным стало только на
|
||||
# post-merge прогоне (#3440).
|
||||
return est._fetch_deals(
|
||||
_db(rows, _BANDS_ROWS if band_rows is None else band_rows),
|
||||
lat=55.74,
|
||||
lon=37.6,
|
||||
rooms=2,
|
||||
area=80.0,
|
||||
radius_m=1000,
|
||||
)
|
||||
|
|
|
|||
Loading…
Add table
Reference in a new issue