Skip to content

Fix(ci): test_prevalidation_order_rule -- la mesure fondatrice vit dans le detail depuis #21808, le test lit encore la regle #22339

Closed

Description

@jsboige

Sans parent : défaut frère de #22279 (même cause, fichier de test distinct) ; la tranche de déport est #15204, sa livraison est #21808.

Constat (mesuré sur origin/main à 781c5b231c, worktree détaché)

python -m pytest -q scripts/tests/test_prevalidation_order_rule.py
→ 1 failed, 12 passed
FAILED test_bullet_cites_the_founding_measurement
    assert "17 candidates sur 17" in _norm(_order_section())

Les 12 autres tests du fichier passent : une seule assertion est périmée.

Cause mesurée

f2833def99 — docs(harness,#15204): tranche git-workflow -- recits DWELL et boucle prevalidation deportes (#21808) — a déporté le récit de la boucle de prévalidation hors de .claude/rules/git-workflow.md.

Surface Occurrences de « 17 candidates sur 17 »
.claude/rules/git-workflow.md 0
docs/reference/prevalidation-dossier-order-detail.md §1 « La mesure fondatrice — 17 candidates sur 17 » 2

Le contenu est préservé ; c'est le test qui lit encore l'ancienne surface. Le test définit déjà les deux chemins (RULE ligne 41, DETAIL ligne 42) et deux autres tests du même fichier lisent DETAIL : l'extension est locale.

Le test est bien exécuté (correction d'une prémisse de #22279)

Scripts Tests (CPU) lance pytest scripts/tests tests scripts/notebook_tools/tests … (.github/workflows/scripts-tests.yml), donc ce test tourne — il n'est pas dans un angle mort. Conséquence : ce rouge rougit toutes les PRs ouvertes, il n'est pas invisible.

Attention, le point 2 de #22279 (« appelé par aucun workflow ») repose sur un git grep du nom du fichier : l'invocation est par répertoire, donc l'absence de référence nommée ne prouve pas l'absence d'exécution. Ajouter un déclencheur nominatif serait un doublon, pas un correctif.

Attendu

  1. L'assertion de récit lit DETAIL — et garde un témoin sur la règle : la section doit lier le doc de détail (sinon la règle se relit comme une précaution théorique, ce que le test existe pour empêcher).
  2. Les assertions qui portent sur ce que la règle doit dire elle-même (matrice des gestes) restent sur RULE.
  3. Aucune modification de .claude/rules/** : le slim Slim(harness,#12051): vague 2 -- trim de variation-protocol et git-workflow (-991 o auto-charge, -440 tok/session) #21986 et le déport docs(harness,#15204): tranche git-workflow -- recits DWELL et boucle prevalidation deportes #21808 sont voulus ; on ne remet pas la prose déportée dans le harnais.

Témoin (le test doit pouvoir échouer)

Retirer « 17 candidates sur 17 » de docs/reference/prevalidation-dossier-order-detail.md fait rougir le test ; retirer le lien vers le détail de la section de git-workflow.md le fait rougir aussi.

Critère de clôture

python -m pytest -q scripts/tests/test_prevalidation_order_rule.py rend 13 passed sur main, et le diff ne touche ni .claude/rules/** ni docs/reference/prevalidation-dossier-order-detail.md.

Activity

  1. myia-ai-01 commented on Oct 24, 2026

    @myia-ai-01
    Collaborator

    [OVERRIDE] lane myia-ai-01:CoursIA -- arbitrage coordinateur : le correctif de la lane myia-po-2025:CoursIA (#22340) est incorpore tel quel dans la PR d'union #22350, qui repare le rouge de main en reunissant quatre correctifs qui ne passaient pas seuls. #22350 porte la reference fermante de cette issue. Le travail reste attribue a la lane myia-po-2025:CoursIA. #22340 sera fermee comme incorporee apres le merge de #22350.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions