Skip to content

[TECH] sinon.restore() entre chaque test dans les fronts - #17146

Open
bpetetot wants to merge 9 commits into
devfrom
refactor/restore-sinon-between-each-tests-in-front
Open

[TECH] sinon.restore() entre chaque test dans les fronts#17146
bpetetot wants to merge 9 commits into
devfrom
refactor/restore-sinon-between-each-tests-in-front

Conversation

@bpetetot

@bpetetot bpetetot commented Aug 14, 2026

Copy link
Copy Markdown
Contributor

☀️ Problème

Contrairement aux tests de l'API, dans les tests frontends, on ne fait pas sinon.restore() entre chaque test.

Or certains tests font des stubs avec sinon qui ne sont jamais restaurés ce qui peut avoir des effets de bord sur les autres tests.

⛱️ Proposition

Pour chaque frontend (mon-pix, admin, orga, certif et junior) :

  • Dans le fichier test-helper.js, ajouter un hook afterEach qui fait sinon.restore()
  • Supprimer les sinon.restore() inutiles dans les tests.

🧴 Remarques

Pas d'impact sur les performances.

🏊 Pour tester

La CI est verte.

@pix-bot-github

Copy link
Copy Markdown

@bpetetot bpetetot self-assigned this Aug 14, 2026
@bpetetot bpetetot added 👀 Tech Review Needed cross-team Toutes les équipes de dev labels Aug 14, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cross-team Toutes les équipes de dev 👀 Tech Review Needed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants