✅ Exige les tests unitaires dans le même développement (raggaroth-senior-dev) #7

New Issue

2026-07-24T10:30:50+01:00

george-claude commented

2026-07-24 10:30:50 +01:00

Consigne : tout développement doit être écrit avec ses tests unitaires — pas ajoutés après coup sur demande séparée.

Constat qui motive ce ticket

Sur scrum-board (refactor #52, découpage de app.py en blueprints), le refactor a été livré et mergé sans suite de tests. Les tests (tests/conftest.py + tests/test_api.py) n'ont été écrits que suite à une demande explicite et séparée ("génère un script de test de bdd"), après coup. Le skill raggaroth-senior-dev ne l'a pas signalé comme manquant avant la livraison.

Constat sur le skill actuel

SKILL.md (section Ancrage méthodes agiles → Definition of Done) mentionne "testé" comme critère à vérifier en review, avant de juger une PR "bon à merger" — mais rien n'impose l'écriture des tests au moment de l'implémentation elle-même, casquette "code" plutôt que casquette "review".

Demande

Mettre à jour raggaroth-senior-dev/SKILL.md pour que la casquette développement (pas seulement code review) porte explicitement cette exigence : toute FEATURE/bugfix/refactor livré(e) doit inclure ses tests unitaires dans le même travail, avant de considérer la tâche terminée — pas une étape a posteriori sur demande séparée de Benjamin. Réviser en cohérence la grille DevSecOps (ligne "Code / Dev") et/ou la section Definition of Done pour que ce soit un réflexe de production, pas seulement un point de contrôle en review.

**Consigne** : tout développement doit être écrit avec ses tests unitaires — pas ajoutés après coup sur demande séparée. **Constat qui motive ce ticket** Sur scrum-board (refactor #52, découpage de `app.py` en blueprints), le refactor a été livré et mergé sans suite de tests. Les tests (`tests/conftest.py` + `tests/test_api.py`) n'ont été écrits que suite à une demande explicite et séparée ("génère un script de test de bdd"), après coup. Le skill `raggaroth-senior-dev` ne l'a pas signalé comme manquant avant la livraison. **Constat sur le skill actuel** `SKILL.md` (section *Ancrage méthodes agiles* → *Definition of Done*) mentionne "testé" comme critère à vérifier **en review**, avant de juger une PR "bon à merger" — mais rien n'impose l'écriture des tests **au moment de l'implémentation** elle-même, casquette "code" plutôt que casquette "review". **Demande** Mettre à jour `raggaroth-senior-dev/SKILL.md` pour que la casquette *développement* (pas seulement *code review*) porte explicitement cette exigence : toute FEATURE/bugfix/refactor livré(e) doit inclure ses tests unitaires dans le même travail, avant de considérer la tâche terminée — pas une étape a posteriori sur demande séparée de Benjamin. Réviser en cohérence la grille DevSecOps (ligne "Code / Dev") et/ou la section Definition of Done pour que ce soit un réflexe de production, pas seulement un point de contrôle en review.

george-claude added the enhancement label 2026-07-24 10:30:50 +01:00

bbaudouin added this to the run skills AI project 2026-07-24 10:31:47 +01:00

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: public/skills-IA#7