Тест-сьют order-fragile: голый pytest -> 17 падений (sys.modules['config'] pollution, пре-существует) #17

New Issue

2026-07-05T10:45:48+03:00

agent_reviewer commented

2026-07-05 10:45:48 +03:00

Найдено при ревью #16, но пре-существует (те же падения на утв. базе стадии-5 d0801ef0).

Тест-сьют order-fragile: голый `pytest` даёт 17 падений

Прогон зависит от порядка коллекции из-за загрязнения sys.modules['config'] тест-моком:

cd tests && pytest        -> 251 passed        (каноничный, зелёный)
pytest tests/  (из корня) -> 251 passed        (как гоняет мейнтейнер)
pytest         (из корня) -> 17 failed, 234 passed
# на базе d0801ef0 (стадия-5): pytest -> 17 failed, 224 passed  => пре-существует

Падения — не логические баги, а async def not natively supported / несостыковки конфига из-за того, что какой-то тест-модуль ставит sys.modules['config'] = tests.mock_config (или подобное) и не восстанавливает, так что при другом порядке коллекции последующие модули видят чужой/мок-конфиг.

CI это не ловит (workflow только собирает docker-образ, pytest не гоняет), и каноничный прогон зелёный — поэтому не блокер. Но новый контрибьютор, запустивший наивный pytest, видит красное.

Fix: изолировать подмену конфига — autouse-фикстура, которая ставит мок в sys.modules и восстанавливает оригинал на teardown (или monkeypatch.setitem(sys.modules, 'config', ...)), чтобы порядок коллекции не влиял. Затрагивает тест-инфру всех стадий, отдельно от продовых изменений.

Найдено при ревью #16, но **пре-существует** (те же падения на утв. базе стадии-5 `d0801ef0`). ## Тест-сьют order-fragile: голый `pytest` даёт 17 падений Прогон зависит от порядка коллекции из-за загрязнения `sys.modules['config']` тест-моком: ``` cd tests && pytest -> 251 passed (каноничный, зелёный) pytest tests/ (из корня) -> 251 passed (как гоняет мейнтейнер) pytest (из корня) -> 17 failed, 234 passed # на базе d0801ef0 (стадия-5): pytest -> 17 failed, 224 passed => пре-существует ``` Падения — не логические баги, а `async def not natively supported` / несостыковки конфига из-за того, что какой-то тест-модуль ставит `sys.modules['config'] = tests.mock_config` (или подобное) и не восстанавливает, так что при другом порядке коллекции последующие модули видят чужой/мок-конфиг. CI это не ловит (workflow только собирает docker-образ, pytest не гоняет), и каноничный прогон зелёный — поэтому не блокер. Но новый контрибьютор, запустивший наивный `pytest`, видит красное. **Fix:** изолировать подмену конфига — autouse-фикстура, которая ставит мок в `sys.modules` и **восстанавливает** оригинал на teardown (или `monkeypatch.setitem(sys.modules, 'config', ...)`), чтобы порядок коллекции не влиял. Затрагивает тест-инфру всех стадий, отдельно от продовых изменений.

agent_reviewer referenced this issue

2026-07-05 10:50:13 +03:00

feat(stability): стадия 6 — лёгкий /ping healthcheck без TG RPC #16

vvzvlad referenced this issue from a commit

2026-07-05 17:20:20 +03:00

fix: escape debug title (XSS) and make test suite order-independent

agent_vscode referenced a pull request that will close this issue

2026-07-05 17:20:33 +03:00

fix: XSS в debug-title (#13) + order-independent тест-сьют (#17) #20

agent_vscode closed this issue

2026-07-05 17:22:03 +03:00

agent_vscode referenced this issue from a commit