bot-backend
80ca7a8a35
All checks were successful
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 / changes (pull_request) Successful in 9s
CI / frontend-tests (pull_request) Has been skipped
CI / openapi-codegen-check (pull_request) Successful in 2m5s
CI / backend-tests (pull_request) Successful in 17m17s
stats["skipped"] был объявлен в контракте, возвращался и печатался в лог — и никогда не увеличивался. Все исходы сваливались в failed: и WAF-блок, и битый разбор, и «UPDATE затронул 0 строк». По такому счётчику нельзя отличить временную помеху от настоящей регрессии разбора, а сам он всегда показывал ноль. Законный источник пропуска в коде БЫЛ: obj_id берутся из БД, но снимок мог смениться между выборкой и UPDATE'ом — тогда строки (obj_id, snapshot_date) уже нет. Ветка с логом «not in DB?» существовала и возвращала False, попадая в failed. Третье состояние сделано через None, а не новым Literal, намеренно: прежние True/False сохраняют смысл, поэтому существующие вызывающие и тесты не переписываются. Проверено прогоном — 531 passed, включая test_domrf_catalog_object_browsersession_throttle (fake возвращает True) и тесты предохранителя из #2971, ни один не тронут. Это и был довод против Literal: менять чужой тест ради своей правки — плохая цена за красоту сигнатуры. Против origin/main: skipped=0, failed=3 вместо skipped=2, failed=1 → падает сумма счётчиков сходится с processed — контроль, зелёный с обеих сторон True/False сохраняют смысл — контроль, зелёный с обеих сторон Первый контроль ловит «починку», при которой пропуск считался бы дважды или терялся; второй фиксирует ровно то свойство, ради которого выбран None. Прогоны: tests/services/scrapers + tests/workers — 531 passed rc=0. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> |
||
|---|---|---|
| .. | ||
| analysis_runs | ||
| cadastre | ||
| chat | ||
| exporters | ||
| forecasting | ||
| generative | ||
| llm | ||
| scrapers | ||
| site_finder | ||
| __init__.py | ||
| test_2464_docs_match_code.py | ||
| test_2464_kn_lock_key.py | ||
| test_2464a_job_settings_savepoint.py | ||
| test_2464h_timing_overlap_formula.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 | ||