[Mineur] Pas de `--timeout` pytest : un test qui hang bloque la CI jusqu'à 360 min #106

New issue

Open

opened 2026-08-26 05:33:47 +00:00 by mat · 0 comments

mat commented

2026-08-26 05:33:47 +00:00

Owner

Problème

pytest.ini ne configure pas --timeout alors que pytest-timeout est déjà disponible (transitif via pytest-homeassistant-custom-component). Un test qui hang (par exemple sur la queue du consumer) bloque le job jusqu'au timeout de la plateforme CI (360 min par défaut).

Recommandation

Ajouter --timeout=120 aux addopts de pytest.ini.

Au passage : quelques sleeps réels dans tests/test_stream.py (asyncio.sleep(0.05/0.1)) sont sources de flakiness faible mais réelle — préférer le pattern polling-borné déjà utilisé ailleurs (20 × 0.05 s).

Constaté lors de la review de code pré-v1.0 (v0.8.1).

## Problème `pytest.ini` ne configure pas `--timeout` alors que `pytest-timeout` est déjà disponible (transitif via pytest-homeassistant-custom-component). Un test qui hang (par exemple sur la queue du consumer) bloque le job jusqu'au timeout de la plateforme CI (360 min par défaut). ## Recommandation Ajouter `--timeout=120` aux `addopts` de `pytest.ini`. Au passage : quelques sleeps réels dans `tests/test_stream.py` (`asyncio.sleep(0.05/0.1)`) sont sources de flakiness faible mais réelle — préférer le pattern polling-borné déjà utilisé ailleurs (20 × 0.05 s). --- Constaté lors de la review de code pré-v1.0 (v0.8.1).

mat added the

v1.0 · mineur

label

2026-08-26 05:33:47 +00:00