Merge pull request 'Красный main после #3440 : тест полос по округам передаёт убранный параметр rooms' (#3453 ) from fix/3051-fetch-deals-rooms-dropped into main

fix(msk): тест полос по округам разъехался с main — у _fetch_deals нет rooms
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
2026-09-11 23:01:32 +00:00 · 2026-09-12 01:53:33 +03:00
1 changed files with 5 additions and 1 deletions
--- a/tradein-mvp/backend/tests/test_3051_moscow_okrug_bands.py
+++ b/tradein-mvp/backend/tests/test_3051_moscow_okrug_bands.py
@ -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,
        )
Author	SHA1	Message	Date
bot-backend	deb6517bd5	Merge pull request 'Красный main после #3440 : тест полос по округам передаёт убранный параметр rooms' (#3453 ) from fix/3051-fetch-deals-rooms-dropped into main All checks were successful Deploy Trade-In / test (push) Successful in 4m43s Details Deploy Trade-In / deploy (push) Successful in 6m17s Details Deploy Trade-In / perimeter-smoke (push) Successful in 1m40s Details Deploy Trade-In / changes (push) Successful in 12s Details Deploy Trade-In / build-frontend (push) Has been skipped Details Deploy Trade-In / build-browser (push) Has been skipped Details Deploy Trade-In / build-backend (push) Successful in 1m6s Details Deploy Trade-In / deploy-status (push) Successful in 1s Details	2026-09-11 23:01:32 +00:00
bot-backend	37494dd74a	fix(msk): тест полос по округам разъехался с main — у _fetch_deals нет rooms All checks were successful CI / changes (pull_request) Successful in 10s Details CI Trade-In / browser-tests (pull_request) Has been skipped Details CI Trade-In / frontend-checks (pull_request) Has been skipped Details CI Trade-In / changes (pull_request) Successful in 9s 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 CI Trade-In / backend-tests (pull_request) Successful in 5m7s Details 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	2026-09-12 01:53:33 +03:00