Skip to content

fix(scripts): verdict_check lit failed_checks avant la sortie brute - #29

Merged
stephrobert merged 1 commit into
mainfrom
fix/verdict-check-failed-checks
Nov 9, 2026
Merged

stephrobert merged 1 commit into
mainfrom
fix/verdict-check-failed-checks

Conversation

@stephrobert

Copy link
Copy Markdown
Owner

Pourquoi

dsoxlab 0.8.0 (stephrobert/dsoxlab#336) retient la sortie brute de check --json pour une épreuve : output vide et output_withheld: true. verdict_check.lire tirait echecs de cette sortie, donc une attestation d'épreuve perdait la liste de ses contrôles ratés (le verdict VALIDE/ROUGE n'était pas affecté).

Ce qui change

  • echecs_du_document lit d'abord failed_checks, présent pour tous les labs depuis 0.8.0 ;
  • repli sur la sortie brute quand le champ est absent (dsoxlab antérieur), mal formé, ou vide alors que le statut est failed.

Fichier identique dans tous les catalogues qui le portent, comme son en-tête l'exige.

Contrôles

  • sept cas éprouvés (0.7.x, lab 0.8.0, épreuve retenue, réussite, liste vide sur échec, liste mal formée, champ nul) ; l'ancienne version échoue sur l'épreuve ;
  • tests existants qui importent le module : ansible-training 27 passés, gitlab-ci-dsoxlab-training 11 passés ;
  • ruff check propre.

Depuis dsoxlab 0.8.0, `check --json` porte `failed_checks` pour tous les labs,
et une épreuve (capstone ou exam_passing_score) ne donne plus rien dans
`output` (vide, `output_withheld: true`). Lire la sortie brute y rendait une
liste d'échecs vide sur une épreuve ratée. Repli sur la sortie brute avant 0.8.0.
@stephrobert
stephrobert merged commit 39679e3 into main Nov 9, 2026
8 checks passed
@stephrobert
stephrobert deleted the fix/verdict-check-failed-checks branch November 9, 2026 15:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant