Repository navigation
fix(test,#13553): le cliquet de compte ne masque plus les invariants de parité - #13602
Merged
Merged
Conversation
…de parite test_full_repo_state_passes_parity : le corps devient _collect_parity_failures (accumulation compte + invariants, sans dependance) ; l'assert finale porte toutes les lignes de defaut. La forme 2 seule ne satisfait pas l'acceptance « les DEUX dans un seul run » — pytest s'arrete a la premiere assertion. 3 tests hermetiques : acceptance, controle negatif (derive seule rouge), etat propre. Contrle negatif execute : simulation du masquage ancient -> test d'acceptance FAIL, restauration -> 35 passed. Co-Authored-By: Claude-Code <noreply@anthropic.com>
This was referenced Aug 30, 2026
github-actions
Bot
added
the
variation-light-cap-reached
Contributor
|
G-VAR-2 light cap reached (advisory, non bloquant). |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Grain: MED/test -- lane myia-po-2026:CoursIA -- prev: MED/tooling #13542
Sujet
test_full_repo_state_passes_parity: l'assertion de compte court-circuite la boucle d'invariants — un échec de compte rend un rouge unique, lisible et rassurant, sans avoir évalué le contenu. See #13553.Contenu (1 fichier)
scripts/translation/tests/test_check_translation_parity.py— le corps du test de référence devient_collect_parity_failures(repo_root, expected_pair_count)qui accumule compte + invariants (lignes de défaut), l'assert finale porte tout. La forme 2 seule (réordonner) ne satisfait pas l'acceptance « les DEUX dans la sortie d'un seul run » (pytest s'arrête à la première assertion) — c'est la forme 1 (accumulation) qui est livrée, sans dépendance (pas de pytest-check/subtests dans l'env).Preuves
python -m pytest scripts/translation/tests/test_check_translation_parity.py -q: 35 passed.python -m pytest scripts/translation/tests -q: 343 passed.python -m pytest scripts/tests -q: 3657 passed, 0 failed.test_13553_count_drift_and_fr_contam_surface_togetherFAIL (1 failure : FR_CONTAM invisible), les 2 autres tests passent ; restauration vérifiée, 35 passed.Contexte
Constat posé en portant #13542 (le geste prescrit par le commentaire du fichier —
EXPECTED_PAIR_COUNT 0 -> 2— a fait ressortir FR_CONTAM que le cliquet masquait). Diagnostic ai-01, suivi demandé par cette lane, feu vert donné dans le body de l'issue. La docstring « Any existing pair ... is still checked against all invariants » redevient vraie dans le seul scénario qu'elle anticipe (reprise du hold i18n).Co-Authored-By: Claude-Code noreply@anthropic.com