fix(tests): объявить 4 live-DB проверки приватности в skip_allowlist
All checks were successful
CI Trade-In / changes (pull_request) Successful in 8s
CI / changes (pull_request) Successful in 8s
CI Trade-In / browser-tests (pull_request) Has been skipped
CI Trade-In / frontend-checks (pull_request) Has been skipped
CI / backend-tests (pull_request) Has been skipped
CI / frontend-tests (pull_request) Has been skipped
CI / openapi-codegen-check (pull_request) Has been skipped
CI Trade-In / backend-tests (pull_request) Successful in 3m46s
All checks were successful
CI Trade-In / changes (pull_request) Successful in 8s
CI / changes (pull_request) Successful in 8s
CI Trade-In / browser-tests (pull_request) Has been skipped
CI Trade-In / frontend-checks (pull_request) Has been skipped
CI / backend-tests (pull_request) Has been skipped
CI / frontend-tests (pull_request) Has been skipped
CI / openapi-codegen-check (pull_request) Has been skipped
CI Trade-In / backend-tests (pull_request) Successful in 3m46s
Хук бюджета пропусков (#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 параллельно, требует строки в списке. Это и есть задуманный обмен — тихий новый пропуск превращается в громкий красный.
This commit is contained in:
parent
d3d7464267
commit
a5582534e3
1 changed files with 9 additions and 0 deletions
|
|
@ -37,3 +37,12 @@ tests/test_house_dedup_merge.py::test_real_fias_pass_ignores_geo_guard
|
|||
tests/test_house_dedup_merge.py::test_real_merge_is_reversible_via_journal
|
||||
tests/test_house_dedup_merge.py::test_real_merge_repoints_dedups_deletes_and_is_idempotent
|
||||
tests/test_user_events.py::test_real_record_event_inserts_row
|
||||
|
||||
# Приватность/ретеншн (#2547) — тот же `_live_session()`. Приехали в main
|
||||
# параллельно с самим списком, поэтому первым же прогоном deploy-лэйна хук их и
|
||||
# поймал: 4 проверки не исполнились и не были объявлены. Проверено против живой
|
||||
# БД (схема из backend/data/sql/ по 233 включительно) — 29 passed, тесты живы.
|
||||
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
|
||||
|
|
|
|||
Loading…
Add table
Reference in a new issue