test(rbac): user2 expected role expired after brusnika trial cutoff #2473

Merged

bot-backend merged 1 commit from fix/rbac-test-user2-expired into main

2026-07-09 16:33:20 +00:00

lekss361 commented

2026-07-09 16:17:16 +00:00

Owner

Что

Follow-up к #2472 (user2 «Брусника» pilot→expired в auth/roles.yaml).

test_get_role_known_users проверял user1..user10 == "pilot" циклом. user2 теперь expired → тест падал: AssertionError: assert 'expired' == 'pilot'.

Почему прошло мимо CI на #2472

На #2472 менялся только auth/roles.yaml (корень репо). Paths-filter ci-tradein.yml / ci.yml матчит tradein-mvp/** / backend/** — тесты были skipped, PR показал зелёный. Провал всплыл уже на полном deploy-tradein (set-all гонит весь suite). Классический false-green от paths-filter.

Фикс

В обоих зеркалах (tradein-mvp/backend/tests/test_rbac.py + backend/tests/test_rbac.py) цикл ожидает expired для n==2, pilot для остальных. Локально test_get_role_known_users — passed.

После мержа этого PR paths-filter матчит backend/** и tradein-mvp/** → тесты реально прогонятся, деплой пройдёт и подхватит user2=expired в рантайме.

## Что Follow-up к #2472 (user2 «Брусника» pilot→expired в `auth/roles.yaml`). `test_get_role_known_users` проверял `user1..user10 == "pilot"` циклом. user2 теперь `expired` → тест падал: `AssertionError: assert 'expired' == 'pilot'`. ## Почему прошло мимо CI на #2472 На #2472 менялся **только** `auth/roles.yaml` (корень репо). Paths-filter `ci-tradein.yml` / `ci.yml` матчит `tradein-mvp/**` / `backend/**` — тесты были **skipped**, PR показал зелёный. Провал всплыл уже на полном `deploy-tradein` (`set-all` гонит весь suite). Классический false-green от paths-filter. ## Фикс В обоих зеркалах (`tradein-mvp/backend/tests/test_rbac.py` + `backend/tests/test_rbac.py`) цикл ожидает `expired` для `n==2`, `pilot` для остальных. Локально `test_get_role_known_users` — passed. После мержа этого PR paths-filter матчит `backend/**` и `tradein-mvp/**` → тесты реально прогонятся, деплой пройдёт и подхватит `user2=expired` в рантайме.

lekss361 added 1 commit 2026-07-09 16:17:17 +00:00

test(rbac): user2 expected role expired after brusnika trial cutoff

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

Details

CI / changes (pull_request) Successful in 10s

Details

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

Details

CI / frontend-tests (pull_request) Has been skipped

Details

CI Trade-In / backend-tests (pull_request) Successful in 1m8s

Details

CI / openapi-codegen-check (pull_request) Successful in 2m33s

Details

CI / backend-tests (pull_request) Successful in 15m22s

Details

f4c26d474a

Follow-up к смене user2 pilot→expired в roles.yaml. test_get_role_known_users
гонял user1..user10 == pilot циклом; user2 теперь expired. На PR #2472 этот
тест был skipped (paths-filter не матчил tradein-mvp/**, менялся только
auth/roles.yaml) → упал уже на полном деплое. Правка в обоих зеркалах
(tradein + main backend).