simplifcation suite a #34 #39

New issue

Closed

opened 2026-03-11 16:15:42 +00:00 by maximus · 0 comments

maximus commented

2026-03-11 16:15:42 +00:00

Owner

Suggestions de simplification

src/hooks/useBudget.test.ts : buildPrevYearTotalMap est une fonction triviale (3 lignes de logique). Les 50 lignes de tests pour une boucle avec un filtre null sont disproportionnées. Un seul test couvrant le cas multi-catégories avec un null suffirait à valider le contrat. Pas bloquant, mais à considérer.
src/hooks/useBudget.ts : buildPrevYearTotalMap est exportée uniquement pour le testing. Vu sa simplicité (3 lignes), elle pourrait rester inline dans useBudget() et être testée indirectement via un test d'intégration du hook. Mais l'approche actuelle est acceptable.

Suggestions de simplification - src/hooks/useBudget.test.ts : buildPrevYearTotalMap est une fonction triviale (3 lignes de logique). Les 50 lignes de tests pour une boucle avec un filtre null sont disproportionnées. Un seul test couvrant le cas multi-catégories avec un null suffirait à valider le contrat. Pas bloquant, mais à considérer. - src/hooks/useBudget.ts : buildPrevYearTotalMap est exportée uniquement pour le testing. Vu sa simplicité (3 lignes), elle pourrait rester inline dans useBudget() et être testée indirectement via un test d'intégration du hook. Mais l'approche actuelle est acceptable.