From a5582534e3fbd79b7616a55a9776b9786c79e42a Mon Sep 17 00:00:00 2001 From: bot-backend Date: Thu, 6 Aug 2026 23:14:25 +0500 Subject: [PATCH] =?UTF-8?q?fix(tests):=20=D0=BE=D0=B1=D1=8A=D1=8F=D0=B2?= =?UTF-8?q?=D0=B8=D1=82=D1=8C=204=20live-DB=20=D0=BF=D1=80=D0=BE=D0=B2?= =?UTF-8?q?=D0=B5=D1=80=D0=BA=D0=B8=20=D0=BF=D1=80=D0=B8=D0=B2=D0=B0=D1=82?= =?UTF-8?q?=D0=BD=D0=BE=D1=81=D1=82=D0=B8=20=D0=B2=20skip=5Fallowlist?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Хук бюджета пропусков (#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 параллельно, требует строки в списке. Это и есть задуманный обмен — тихий новый пропуск превращается в громкий красный. --- tradein-mvp/backend/tests/skip_allowlist.txt | 9 +++++++++ 1 file changed, 9 insertions(+) diff --git a/tradein-mvp/backend/tests/skip_allowlist.txt b/tradein-mvp/backend/tests/skip_allowlist.txt index 5c0b9a20..02f7d075 100644 --- a/tradein-mvp/backend/tests/skip_allowlist.txt +++ b/tradein-mvp/backend/tests/skip_allowlist.txt @@ -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 -- 2.45.3