Найдено поиском «правило соблюдено рядом — нарушено здесь». В самой функции
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
