Claude commited on
Commit
31ef91a
·
unverified ·
1 Parent(s): 0b09377

fix(tests): docstring rot dans test_chantier5.py post-découpage runner

Browse files

Suite à l'audit du commit 0b09377 (découpage de runner.py en sous-package).
Le docstring de ``TestRunnerStillReachable`` (section 5.C) affirmait :
« Le chantier 5 ne le découpe pas davantage » — ce qui était vrai au
moment du chantier 5 mais devenu faux après le sprint « découpage de
runner.py » (mai 2026).

Les assertions du test passent toujours via les ré-exports du sous-package,
mais le docstring trahissait la réalité. Reformulé pour décrire l'invariant
réel (l'API historique reste accessible) sans dépendre de la forme physique
du module (fichier unique vs sous-package).

Trouvé par audit Explore récursif post-commit.

https://claude.ai/code/session_018us43uphCvqwm2TARqyYoH

tests/integration/test_chantier5.py CHANGED
@@ -213,14 +213,18 @@ class TestCliPackage:
213
 
214
 
215
  # ──────────────────────────────────────────────────────────────────────────
216
- # 5.C — runner.py reste tel quel (déjà allégé au chantier 2)
217
  # ──────────────────────────────────────────────────────────────────────────
218
 
219
 
220
  class TestRunnerStillReachable:
221
- """Le chantier 2 a déjà allégé ``runner.py`` de 303 lignes (1322 → 1019).
222
- Le chantier 5 ne le découpe pas davantage — vérification que les
223
- fonctions historiques restent accessibles."""
 
 
 
 
224
 
225
  @pytest.mark.parametrize("name", [
226
  "run_benchmark",
 
213
 
214
 
215
  # ──────────────────────────────────────────────────────────────────────────
216
+ # 5.C — runner reste atteignable via son API publique historique
217
  # ──────────────────────────────────────────────────────────────────────────
218
 
219
 
220
  class TestRunnerStillReachable:
221
+ """L'API historique de ``picarones.measurements.runner`` reste accessible.
222
+
223
+ Le chantier 2 (post-Sprint 97) avait allégé ``runner.py`` de 303 lignes
224
+ (1322 → 1019) ; le sprint « découpage de runner.py » (mai 2026, hors
225
+ chantier 5) l'a ensuite éclaté en sous-package ``runner/``. Dans tous
226
+ les cas, les fonctions historiques restent atteignables via les
227
+ ré-exports — c'est ce qu'on vérifie ici."""
228
 
229
  @pytest.mark.parametrize("name", [
230
  "run_benchmark",