fix(tests): объявить два живых теста #3063 в skip_allowlist — main красный, деплой встал #3069

Merged
lekss361 merged 1 commit from fix/3063-skip-allowlist into main 2026-08-23 22:14:50 +00:00

1 commit

Author SHA1 Message Date
bot-backend
db3b58ee3e fix(tests): объявить два живых теста #3063 в skip_allowlist
All checks were successful
CI Trade-In / browser-tests (pull_request) Has been skipped
CI Trade-In / changes (pull_request) Successful in 7s
CI / changes (pull_request) Successful in 9s
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 4m29s
Main красный: `Deploy Trade-In / test` падает, хотя pytest прошёл
(4732 passed, 34 skipped). Валит его гард из tests/conftest.py:

  НЕУЧТЁННЫЙ ПРОПУСК (2): проверка не исполнилась и не объявлена
  в skip_allowlist.txt

Гард сработал ровно как задуман — «пропуск без записи неотличим от
пройденной проверки» (#2722/#2729/#2740). Два поведенческих теста из
#3067 требуют живую Postgres: в ci-tradein.yml есть postgres-сервис и
они реально исполняются (там PR и был зелёным), а в deploy-tradein.yml
БД нет вовсе — отсюда зелёный PR и красный main.

Из-за этого три скрапперных правки (#3065, #3067, #3068) собрались, но
НЕ выкатились — деплой встал на тест-гейте.

Запись объясняет, почему проверку нельзя выполнить в deploy-лэйне и где
она выполняется вместо этого, как требует шапка файла. Статическая
половина того же файла (гард COALESCE + гард паритета кортежей) БД не
требует и идёт на обоих лэйнах — то есть без записи остаётся не дыра, а
только поведенческая проверка.

Проверено локально фальсификацией: со старым allowlist гард печатает
«НЕУЧТЁННЫЙ ПРОПУСК (2)», с новым молчит; тесты в обоих случаях
7 passed, 2 skipped.

Refs #3063, #3067
2026-08-24 01:07:26 +03:00