bot-backend
b9381e15e0
All checks were successful
CI / changes (pull_request) Successful in 10s
CI / frontend-tests (pull_request) Has been skipped
CI Trade-In / changes (pull_request) Successful in 8s
CI Trade-In / backend-tests (pull_request) Has been skipped
CI Trade-In / browser-tests (pull_request) Has been skipped
CI Trade-In / frontend-checks (pull_request) Has been skipped
CI / openapi-codegen-check (pull_request) Successful in 2m41s
CI / backend-tests (pull_request) Successful in 17m29s
Найдено поиском «правило соблюдено рядом — нарушено здесь». В самой функции
analyze_parcel двенадцать соседних блоков оборачивают свой db.execute в
db.begin_nested(), и у velocity-блока правило записано словами:
«SAVEPOINT защищает outer transaction если velocity SQL падает — иначе
следующие queries (_geotech_risk и пр.) крашатся с InFailedSqlTransaction»
Два блока из правила выпадали:
1. get_permits_nearby — вызов на общей сессии, своей защиты не имеет, ошибку
глотает вызывающий. Разрыв с ближайшим честным соседом — 60 строк в ОДНОЙ
функции.
2. _neighbors_summary — тот же класс, вызывается перед записью прогона.
Радиус поражения проверен по коду: после незащищённого блока разрешений идут
_geotech_risk (:4141), _neighbors_summary (:4145) и persist_analysis_run (~:4220).
Один упавший запрос к gisogd_permits оставлял транзакцию в aborted-состоянии и
уносил ВЕСЬ анализ, а не блок разрешений.
Тест использует двойник сессии из #2938 (tests/support/pg_session.py) —
MagicMock aborted-состояния не имеет, и на нём проверка «сессия осталась рабочей»
зелена независимо от наличия SAVEPOINT. Красный на origin/main:
`AbortedTransactionError: current transaction is aborted`.
ЧЕСТНО ПРО ПОКРЫТИЕ: прямой тест есть только на _neighbors_summary — это отдельная
функция. Блок get_permits_nearby лежит внутри обработчика, дотянуться можно лишь
через полный эндпоинт. Первая версия файла «покрывала» его тестом, который САМ
писал `with db.begin_nested():` вокруг вызова и проверял собственную обёртку —
такая проверка зелена независимо от боевого кода. Тест удалён, ограничение
записано в шапке: лучше честно отсутствующее покрытие, чем поддельное.
pytest tests/api/v1: 357 passed, 1 skipped, rc=0
|
||
|---|---|---|
| .. | ||
| alembic | ||
| app | ||
| db/init | ||
| output | ||
| scripts | ||
| tests | ||
| .dockerignore | ||
| .env.example | ||
| .env.runtime.example | ||
| .gitignore | ||
| alembic.ini | ||
| debug.log | ||
| Dockerfile | ||
| pyproject.toml | ||
| uv.lock | ||