ci(tradein): снять последний deselect — он пережил починку своего теста #2756

Merged

bot-backend merged 1 commit from chore/drop-stale-deselect into main

2026-08-06 20:53:55 +00:00

bot-backend commented

2026-08-06 20:52:09 +00:00

Collaborator

Summary

Последний --deselect в репозитории. Жил в post-merge test-job'е deploy-tradein.yml с объяснением, которое было неверным в обеих половинах: «падает ТОЛЬКО в whole-suite ordering, в изоляции проходит — global-state leak из другого модуля».

На деле тест падал и в изоляции: он ходил в /api/v1/search без заголовка X-Authenticated-User, а RBAC-гард отвечает на такое 401. Причина была в самом тесте. Заголовок добавлен в #2729, и в pre-merge гейте (ci-tradein.yml) строка снята тогда же — здесь она пережила починку своего теста ещё на месяц, потому что файл был занят открытым #2680. Тот смержен (d3d74642), долг закрыт.

Заодно -rs, как в остальных лэйнах.

После этого PR в репозитории не остаётся ни одного --deselect.

Test plan

тест зелёный в полном прогоне: ci-tradein гоняет его без deselect'а с #2729, все прогоны зелёные (последний — 3962 passed)
YAML парсится
после мержа: post-merge Deploy Trade-In / test без строки 1 deselected, 13 объявленных пропусков

Refs #2722 #2729 #2745

## Summary Последний `--deselect` в репозитории. Жил в post-merge test-job'е `deploy-tradein.yml` с объяснением, которое было неверным в обеих половинах: «падает ТОЛЬКО в whole-suite ordering, в изоляции проходит — global-state leak из другого модуля». На деле тест падал и в изоляции: он ходил в `/api/v1/search` без заголовка `X-Authenticated-User`, а RBAC-гард отвечает на такое 401. Причина была в самом тесте. Заголовок добавлен в #2729, и в pre-merge гейте (`ci-tradein.yml`) строка снята тогда же — **здесь она пережила починку своего теста ещё на месяц**, потому что файл был занят открытым #2680. Тот смержен (d3d74642), долг закрыт. Заодно `-rs`, как в остальных лэйнах. **После этого PR в репозитории не остаётся ни одного `--deselect`.** ## Test plan - [x] тест зелёный в полном прогоне: `ci-tradein` гоняет его без deselect'а с #2729, все прогоны зелёные (последний — 3962 passed) - [x] YAML парсится - [ ] после мержа: post-merge `Deploy Trade-In / test` без строки `1 deselected`, 13 объявленных пропусков Refs #2722 #2729 #2745

bot-backend added 1 commit 2026-08-06 20:52:09 +00:00

ci(tradein): снять последний deselect — он пережил починку своего теста

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

Details

CI / changes (pull_request) Successful in 8s

Details

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

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

978461e96a

`--deselect tests/test_search_api.py::test_search_cache_hit` в post-merge
test-job'е остался единственным deselect'ом в репозитории. Его объяснение
(«падает только при полном прогоне, в изоляции проходит — global-state leak»)
было неверным в обеих половинах: тест падал и в изоляции, потому что ходил в
/api/v1/search без X-Authenticated-User, а RBAC-гард отвечает 401. Причина
была в тесте; заголовок добавлен в #2729, и в pre-merge гейте строка снята
тогда же. Здесь она пережила починку ещё на месяц — файл был занят открытым
#2680. Тот смержен, долг закрыт.

Заодно `-rs`, как в остальных лэйнах: пропуск печатает причину. Ожидание в
этом лэйне — 13 объявленных пропусков; неучтённый роняет прогон через хук
бюджета (#2745).

После этого коммита в репозитории не остаётся ни одного `--deselect`.