bot-backend
7e21ff2af9
All checks were successful
CI Trade-In / changes (pull_request) Successful in 17s
CI / changes (pull_request) Successful in 17s
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 2m58s
CI Trade-In / backend-tests (pull_request) Successful in 5m15s
CI / backend-tests (pull_request) Successful in 16m3s
Оба теста красили ЧУЖИЕ PR-ы по таймингу раннера, а на правиле «зелёный CI = можно мержить» здесь держится весь самомерж. Диагнозы разные, поэтому и правки разные. 1) tradein auth-флуд (`test_login_flood_capped_by_rate_…`). Утверждение про потолок верное — подводило ПРЕДУСЛОВИЕ `len(codes) > connections`: «каждое из ста соединений успело сходить дважды за секунду» мерит скорость раннера, а не нагрузку. На занятом первый круг из ста запросов сам съедает всю секунду, и невыполненное предусловие давало красный, неотличимый от настоящей поломки потолка. Замер: свободная машина — 0 падений из 20 (~800 ответов/с), под load average ~185 — 10 из 15, и в каждом флуд всё равно предлагал 55-80 запросов/с против порога 30/с, то есть мерить БЫЛО на чем. Теперь предусловие содержательное и с тем же порогом, с которым сверяется вердикт: пока флуд предлагает меньше `ceiling × 1.5`, следующее утверждение зелено даже без потолка вовсе — значит измерения нет, и красный говорит «раннер не потянул», а не «потолок сломан». Остальные четыре утверждения не тронуты. 2) backend weather_cache. Здесь подводило САМО утверждение: тест требовал «ровно один сетевой вызов на 16 потоков», а код (#1370) прямым текстом выносит вызов за lock и обещает обратное — «может породить несколько параллельных запросов… приемлемо». Зелёным он был по везению GIL: 200 штормов дали 2 вызова один раз (~0.5% — этим и покрасило #2781), а при `setswitchinterval(1e-6)` больше одного вызова дали 197 штормов из 200. Тест приведён к тому, что код правда гарантирует: значение у всех потоков одно, сетевых вызовов не больше числа участников, ключ один, и ПОСЛЕ шторма кэш отвечает без сети. Код не тронут: поведение объявлено приемлемым в #1370 с обоснованием; настоящий single-flight (per-key lock) — отдельная задача с отдельным обоснованием, а не побочный эффект правки теста. Оба сторожа проверены на способность краснеть: снятый гейт насыщения + пул мимо настройки → «957 сверок/с при потолке 20/с» (и под нагрузкой тоже краснеет, на том же прогоне, где раньше падало предусловие); потерянная запись в кэш / чужой ключ / мёртвый TTL → три разных красных сообщения. Refs #2783 |
||
|---|---|---|
| .. | ||
| api | ||
| fixtures | ||
| integration | ||
| scrapers | ||
| scripts | ||
| services | ||
| smoke | ||
| sql | ||
| workers | ||
| __init__.py | ||
| conftest.py | ||
| skip_allowlist.txt | ||
| test_admin_weight_profiles.py | ||
| test_app_logging.py | ||
| test_audit_middleware.py | ||
| test_auth_db.py | ||
| test_auth_session_guard.py | ||
| test_auth_session_service.py | ||
| test_concepts_stub.py | ||
| test_connection_capacity_loaders.py | ||
| test_eesk_reserve_loader.py | ||
| test_eias_heat_loader.py | ||
| test_ekb_geoportal_client.py | ||
| test_ekgas_outlet_loader.py | ||
| test_gas_grs_loader.py | ||
| test_gate_verdict.py | ||
| test_gisogd66_loader.py | ||
| test_health.py | ||
| test_layout_signature.py | ||
| test_layout_tz_html.py | ||
| test_layout_tz_pdf.py | ||
| test_network_obremenenie.py | ||
| test_network_zone_humanize.py | ||
| test_noise_loader.py | ||
| test_nspd_client.py | ||
| test_nspd_sync.py | ||
| test_ors_client.py | ||
| test_osm_tp_geocoder.py | ||
| test_osrm_client_local.py | ||
| test_ping.py | ||
| test_poi_score.py | ||
| test_quarter_dump_lookup.py | ||
| test_rbac.py | ||
| test_riasurt_sverdl_client.py | ||
| test_saturation.py | ||
| test_sentry_init.py | ||
| test_utility_infrastructure_loader.py | ||
| test_velocity.py | ||
| test_weight_profiles.py | ||