Repository navigation
[tooling] 4 paires de tests byte-identiques dans test_check_self_hosted_runner_policy.py (2e def éclipse la 1re, l.460-555) #13960
Description
Constat
scripts/tests/test_check_self_hosted_runner_policy.py (main, 597 lignes) porte 4 paires de définitions de fonctions byte-identiques — la 2e définition éclipse la 1re en Python :
| Fonction | def 1 | def 2 | byte-identiques |
|---|---|---|---|
test_missing_trigger_breaks_instrument |
l.460-470 | l.509-519 | ✅ (sha256 b4df2c63…) |
test_empty_runs_on_list_breaks_instrument |
l.473-486 | l.522-535 | ✅ (sha256 5729c27b…) |
test_missing_jobs_breaks_instrument |
l.489-495 | l.538-544 | ✅ (sha256 9977b325…) |
test_same_repository_reusable_workflow_rejects_path_traversal |
l.498-506 | l.547-555 | ✅ (sha256 3271a663…) |
Pourquoi c'est un problème
- Sans effet aujourd'hui : les deux corps sont identiques, donc l'éclipse est neutre en couverture.
- Code mort + test fantôme demain : si une des deux copies évolue (ou une seule est corrigée lors d'un futur refactoring), la sémantique du fichier change silencieusement — la version visible dans un éditeur n'est pas nécessairement celle exécutée.
- Signé aussi par la confusion possible lors de tout futur
git blame/review sur ce bloc.
Vérification (2026-09-01, Hermes po-2026)
- Fichier fetché depuis
main(raw.githubusercontent + cross-check API), parsé viaast: 4 noms en double au niveau module, segments comparés ligne-à-ligne → identiques. - Trouvé pendant la review fix(ci,#13874): forme universelle de la garde fork -- couvre pull_request_target #13947 (fix(ci,ci(#13378): la garde fork du runner Linux laisse passer pull_request_target #13874) garde fork universelle — note mineure de la review, hors scope de cette PR), vérifié indépendamment pour cette issue.
- Pas de doublon avec Test file duplication: 12 files in scripts/tests/ shadow canonical homes (caused #10055 CI break) #10066 (duplication de fichiers entre répertoires — autre classe).
Fix suggéré
One-liner de dédup : supprimer une des deux copies de chaque paire (l.460-506 ou l.509-555 — indifférent, les corps sont identiques). Grain LIGHT, aucun changement de comportement attendu (pytest scripts/tests/test_check_self_hosted_runner_policy.py doit rester vert au même nombre de tests).
Activity
fix(tests,#13960): remove 4 byte-identical duplicate test definitions (…
fix(tooling,#13960): dedup 4 byte-identical test functions + AST guard (
jsboige commentedon Sep 27, 2026
Verification G.9 (tranche 1 de #15258, 27/09) — fermeture legitime, attribution :
b6903955a0(2026-09-02T00:13:24Z, mot-cle Close) = PR fix(tests,#13960): remove 4 byte-identical duplicate test definitions #13987 MERGEDfix(tests,#13960) : remove 4 byte-identical duplicate test de...— l'objet (les 4 paires de definitions byte-identiques) est la substance meme de la PR.Grain: LIGHT/test -- lane myia-po-2026:CoursIA.Aucune reouverture requise.