bot-backend
4c0abb8a0d
All checks were successful
CI Trade-In / changes (pull_request) Successful in 8s
CI / changes (pull_request) Successful in 10s
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 / frontend-tests (pull_request) Has been skipped
CI / openapi-codegen-check (pull_request) Successful in 2m10s
CI / backend-tests (pull_request) Successful in 17m14s
get_all/get_one глотают ошибку db.execute и возвращают fallback, но сессию им отдаёт вызывающий: admin-ручки, beat_schedule, и get_setting_value из cadastre_fetch/nspd_geo. На Postgres упавший execute оставляет транзакцию в aborted-состоянии — все последующие запросы ЭТОЙ ЖЕ сессии падают с «current transaction is aborted», то есть падает не тот, кто виноват. Голый db.rollback() здесь запрещён: он снёс бы незакоммиченную работу вызывающего. Средство — SAVEPOINT вокруг самого execute, как в уже закрытом пункте того же кластера (developer_attribution.py:152). Мок в тесте ВОСПРОИЗВОДИТ семантику Postgres: упавший запрос переводит сессию в aborted, дальнейшие падают, откат SAVEPOINT восстанавливает. Это не педантизм. Существующий образец в tests/test_saturation.py устроен иначе — begin_nested там пустой контекст-менеджер без aborted-состояния, — и проверка на отравление проходит независимо от наличия защиты. Проверено экспериментом: со СНЯТЫМ SAVEPOINT в saturation.py все 19 тестов файла зелёные. Тест, который не может покраснеть при снятии охраняемого, охраны не проверяет. Поэтому здесь есть ещё и контроль на сам мок (test_mock_actually_poisons_ without_a_savepoint): без него проверки были бы зелёными по построению. Тесты: 2 красных на origin/main с настоящим текстом ошибки Postgres («current transaction is aborted, commands ignored until end of transaction block»), 2 контроля зелёные с обеих сторон. pytest tests/services: 3063 passed, 14 skipped, rc=0 |
||
|---|---|---|
| .. | ||
| analysis_runs | ||
| cadastre | ||
| chat | ||
| exporters | ||
| forecasting | ||
| generative | ||
| llm | ||
| scrapers | ||
| site_finder | ||
| __init__.py | ||
| test_2464a_job_settings_savepoint.py | ||
| test_analytics_queries_domrf_dedup.py | ||
| test_cadastre_bulk.py | ||
| test_dadata_client.py | ||
| test_ekburg_permits.py | ||
| test_forecast_request_cache.py | ||
| test_functional_zone_lookup.py | ||
| test_granddoc_lookup.py | ||
| test_ird_analyze.py | ||
| test_ird_overlay_lookup.py | ||
| test_job_settings.py | ||
| test_krt_lookup.py | ||
| test_mapping_review.py | ||
| test_newbuilding_crossload.py | ||
| test_nspd_denorm.py | ||
| test_obj_class_backfill.py | ||
| test_objective_backfill.py | ||
| test_own_developer_ids_config.py | ||
| test_planning_lookup.py | ||
| test_ppt_tep_lookup.py | ||
| test_quarter_dump_lookup.py | ||
| test_recommend_mix_velocity.py | ||
| test_reservation_lookup.py | ||
| test_utility_characteristics.py | ||
| test_weather_cache.py | ||
| test_zone_regulation_backfill.py | ||
| test_zone_regulation_extract.py | ||
| test_zone_regulation_memo.py | ||
| test_zone_regulation_write_session.py | ||