fix(tests): объявить 4 live-DB проверки приватности в skip_allowlist #2750

Merged

bot-backend merged 1 commit from chore/skip-allowlist-privacy into main

2026-08-06 18:20:05 +00:00

bot-backend commented

2026-08-06 18:14:56 +00:00

Collaborator

Summary

Хук бюджета пропусков (#2745) и тесты приватности (#2547) приехали в main почти одновременно — и первым же прогоном deploy-лэйна хук поймал ровно то, для чего сделан:

НЕУЧТЁННЫЙ ПРОПУСК (4): проверка не исполнилась и не объявлена в skip_allowlist.txt:
  - tests/test_data_erasure.py::test_real_erase_by_phone_does_not_match_different_number
  - tests/test_data_erasure.py::test_real_erase_by_phone_finds_differently_formatted_number
  - tests/test_data_erasure.py::test_real_erase_by_phone_finds_ru_trunk_prefix_variant
  - tests/test_purge_expired_trade_in_data.py::test_real_purge_deletes_only_anonymous_expired_estimates

Они на том же _live_session(), что и остальная группа: в pre-merge гейте ci-tradein исполняются (там поднят Postgres), в deploy-лэйне без БД — пропускаются. Поэтому объявление в списке, а не починка.

Тесты живы, с кодом не разошлись — проверено против живой Postgres со схемой из backend/data/sql/ по 233 включительно: 29 passed.

Заодно видна цена механизма: тест с условным пропуском, приехавший в main параллельно, требует строки в списке. Это и есть задуманный обмен — тихий новый пропуск превращается в громкий красный.

Test plan

против живой БД: tests/test_data_erasure.py tests/test_purge_expired_trade_in_data.py → 29 passed
в лэйне без БД (как в deploy): полный сьют → 3946 passed / 14 skipped, хук молчит, exit 0

Refs #2745 #2547

## Summary Хук бюджета пропусков (#2745) и тесты приватности (#2547) приехали в main почти одновременно — и первым же прогоном deploy-лэйна хук поймал ровно то, для чего сделан: ``` НЕУЧТЁННЫЙ ПРОПУСК (4): проверка не исполнилась и не объявлена в skip_allowlist.txt: - tests/test_data_erasure.py::test_real_erase_by_phone_does_not_match_different_number - tests/test_data_erasure.py::test_real_erase_by_phone_finds_differently_formatted_number - tests/test_data_erasure.py::test_real_erase_by_phone_finds_ru_trunk_prefix_variant - tests/test_purge_expired_trade_in_data.py::test_real_purge_deletes_only_anonymous_expired_estimates ``` Они на том же `_live_session()`, что и остальная группа: в pre-merge гейте `ci-tradein` исполняются (там поднят Postgres), в deploy-лэйне без БД — пропускаются. Поэтому объявление в списке, а не починка. **Тесты живы, с кодом не разошлись** — проверено против живой Postgres со схемой из `backend/data/sql/` по 233 включительно: **29 passed**. Заодно видна цена механизма: тест с условным пропуском, приехавший в main параллельно, требует строки в списке. Это и есть задуманный обмен — тихий новый пропуск превращается в громкий красный. ## Test plan - [x] против живой БД: `tests/test_data_erasure.py tests/test_purge_expired_trade_in_data.py` → 29 passed - [x] в лэйне без БД (как в deploy): полный сьют → 3946 passed / 14 skipped, хук молчит, exit 0 Refs #2745 #2547

bot-backend added 1 commit 2026-08-06 18:14:57 +00:00

fix(tests): объявить 4 live-DB проверки приватности в skip_allowlist

CI Trade-In / changes (pull_request) Successful in 8s

Details

CI / changes (pull_request) Successful in 8s

Details

CI Trade-In / browser-tests (pull_request) Has been skipped

Details

CI Trade-In / frontend-checks (pull_request) Has been skipped

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

CI Trade-In / backend-tests (pull_request) Successful in 3m46s

Details

a5582534e3

Хук бюджета пропусков (#2745) и тесты приватности (#2547) приехали в main
почти одновременно, и первым же прогоном deploy-лэйна хук поймал ровно то, для
чего сделан: четыре проверки не исполнились и не были объявлены —
test_data_erasure (3) и test_purge_expired_trade_in_data (1). Они на том же
`_live_session()`, что и остальная группа: в pre-merge гейте ci-tradein
исполняются (там поднят Postgres), в deploy-лэйне без БД — пропускаются.

Проверено против живой БД со схемой из backend/data/sql/ по 233 включительно:
29 passed, тесты живы и с кодом не разошлись. Поэтому именно объявление в
списке, а не починка.

Заодно видно цену механизма: тест с условным пропуском, приехавший в main
параллельно, требует строки в списке. Это и есть задуманный обмен — тихий
новый пропуск превращается в громкий красный.