Красный main после #3440: тест полос по округам передаёт убранный параметр rooms #3453

Merged

bot-backend merged 1 commit from fix/3051-fetch-deals-rooms-dropped into main

2026-09-11 23:01:33 +00:00

bot-backend commented

2026-09-11 22:54:00 +00:00

Collaborator

Post-merge прогон main после #3440 дал 5 failed, 5888 passed: TypeError: _fetch_deals() got an unexpected keyword argument 'rooms' в tests/test_3051_moscow_okrug_bands.py.

Это семантический конфликт мержа, а не поломка. Ветка полос по округам отводилась до #3256, который убрал параметр rooms у _fetch_deals — сделки Росреестра комнатность не несут, и аргумент «на будущее» только приглашал вернуть предикат, фильтрующий на самом деле по площади. Текстового конфликта git не увидел: одна сторона правила сигнатуру, другая — тест. Pre-merge CI ветки был зелёным именно поэтому.

Эффекта в проде нет. _fetch_deals не вызывается из app/ ни разу — ни до мержа, ни после, функцию держат только эти тесты. Проверено git grep по трём деревьям: main до мержа, ветка, main после.

Проверено

uv run python -m pytest tests/test_3051_moscow_okrug_bands.py — 24 passed
uv run ruff check . — All checks passed

🤖 Generated with Claude Code

https://claude.ai/code/session_01VQ8jqr4SFirX5tFLwdSrXh

Post-merge прогон main после #3440 дал `5 failed, 5888 passed`: `TypeError: _fetch_deals() got an unexpected keyword argument 'rooms'` в `tests/test_3051_moscow_okrug_bands.py`. Это семантический конфликт мержа, а не поломка. Ветка полос по округам отводилась до #3256, который **убрал** параметр `rooms` у `_fetch_deals` — сделки Росреестра комнатность не несут, и аргумент «на будущее» только приглашал вернуть предикат, фильтрующий на самом деле по площади. Текстового конфликта git не увидел: одна сторона правила сигнатуру, другая — тест. Pre-merge CI ветки был зелёным именно поэтому. **Эффекта в проде нет.** `_fetch_deals` не вызывается из `app/` ни разу — ни до мержа, ни после, функцию держат только эти тесты. Проверено `git grep` по трём деревьям: main до мержа, ветка, main после. ## Проверено - `uv run python -m pytest tests/test_3051_moscow_okrug_bands.py` — 24 passed - `uv run ruff check .` — All checks passed 🤖 Generated with [Claude Code](https://claude.com/claude-code) https://claude.ai/code/session_01VQ8jqr4SFirX5tFLwdSrXh

bot-backend added 1 commit 2026-09-11 22:54:00 +00:00

fix(msk): тест полос по округам разъехался с main — у _fetch_deals нет rooms

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

37494dd74a

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