test(tradein): cover avito/cian + mark_failed total_seen persistence (#1926) #1937

Merged

bot-backend merged 1 commit from fix/1926-scrape-runs-total-seen into main

2026-06-27 00:17:24 +00:00

bot-backend commented

2026-06-27 00:16:29 +00:00

Collaborator

Что

Тест-хардинг для уже смерженного фикса observability scrape_runs.total_seen/new_count (#1928, _column_counts).

#1928 починил персист total_seen/new_count из counters во ВСЕ sweep-пути (через единый _column_counts в scrape_runs.py: update_heartbeat/mark_done/mark_failed/mark_banned), но regression-тест добавил только на yandex mark_done. Аудит #1926 явно просил «проверить avito/cian».

Добавлено (+49 строк, только тесты)

test_mark_done_avito_cian_counters_populate_columns — CitySweepCounters.to_dict() (avito/cian) → mark_done ставит total_seen=1234, new_count=56 (не 0).
test_mark_failed_persists_total_seen_new_count_columns — sweep упавший mid-run всё равно пишет total_seen/new_count из частичных counters (не 0).

Проверки

ruff check ✅ · full suite 2446 passed, 2 skipped (CI-deselects).
Чисто тесты — функциональных изменений нет (фикс уже в проде с #1928).

Refs #1926

## Что Тест-хардинг для уже смерженного фикса observability `scrape_runs.total_seen/new_count` (#1928, `_column_counts`). #1928 починил персист `total_seen`/`new_count` из counters во ВСЕ sweep-пути (через единый `_column_counts` в `scrape_runs.py`: `update_heartbeat`/`mark_done`/`mark_failed`/`mark_banned`), но regression-тест добавил только на **yandex `mark_done`**. Аудит #1926 явно просил «проверить avito/cian». ## Добавлено (+49 строк, только тесты) - `test_mark_done_avito_cian_counters_populate_columns` — `CitySweepCounters.to_dict()` (avito/cian) → `mark_done` ставит `total_seen=1234`, `new_count=56` (не 0). - `test_mark_failed_persists_total_seen_new_count_columns` — sweep упавший mid-run всё равно пишет `total_seen`/`new_count` из частичных counters (не 0). ## Проверки - `ruff check` ✅ · full suite **2446 passed, 2 skipped** (CI-deselects). - Чисто тесты — функциональных изменений нет (фикс уже в проде с #1928). Refs #1926

bot-backend added 1 commit 2026-06-27 00:16:30 +00:00

test(tradein): cover avito/cian + mark_failed total_seen/new_count persistence (#1926 )

CI / changes (pull_request) Successful in 7s

Details

CI / backend-tests (pull_request) Has been skipped

Details

CI / frontend-tests (pull_request) Has been skipped

Details

CI / openapi-codegen-check (pull_request) Has been skipped

Details

8bb302aadf

#1928 fixed scrape_runs.total_seen/new_count for sweeps via _column_counts
but only regression-tested the yandex mark_done path. Add equivalents for
the CitySweepCounters (avito/cian) sweep and the mark_failed path — both
explicitly named in the #1926 audit — so the all-sweeps + failure-path
guarantee is locked in.

Refs #1926