Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[TECH] Réécrire les tests unitaires du usecase authenticateUser en tests d'intégration (PIX-16757) #11845

Conversation

lego-technix
Copy link
Contributor

@lego-technix lego-technix commented Mar 25, 2025

🌸 Problème

Comme tous les tests unitaires de usecases, le usecase api/tests/identity-access-management/unit/domain/usecases/authenticate-user_test.js teste plus l’implémentation qu’autre chose et est très difficile à faire évoluer. C’est très très pénible lorsqu’on doit faire évoluer ce usecase, ce que l’on fait régulièrement et qu’on sera encore amené à faire régulièrement dans le futur.

🌳 Proposition

Conformément aux standards de tests, réécrire api/tests/identity-access-management/unit/domain/usecases/authenticate-user_test.js sous la forme de tests d'intégration api/tests/identity-access-management/integration/domain/usecases/authenticate-user.usecase.test.js.

On a profité de cette réécriture pour mieux agencer et regrouper (avec des describe et des context) les tests, et pour mieux identifier les différents tests.

🐝 Remarques

RAS

🤧 Pour tester

  1. Vérifier que la CI passe,
  2. Vérifier que les nouveaux tests sont bien agencés, regroupés, identifiés et qu'ils couvrent bien tous les cas.

@lego-technix lego-technix requested a review from a team as a code owner March 25, 2025 18:43
@pix-bot-github
Copy link

Une fois les applications déployées, elles seront accessibles via les liens suivants :

Les variables d'environnement seront accessibles via les liens suivants :

Copy link
Contributor

@bpetetot bpetetot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Super boulot 👍

@pix-service-auto-merge pix-service-auto-merge force-pushed the pix-16757-rewrite-authenticateUser-usecase-tests-as-integration-tests branch from a2fb400 to ac5610a Compare March 27, 2025 13:10
@pix-service-auto-merge pix-service-auto-merge merged commit 85c3f77 into dev Mar 27, 2025
9 of 11 checks passed
@pix-service-auto-merge pix-service-auto-merge deleted the pix-16757-rewrite-authenticateUser-usecase-tests-as-integration-tests branch March 27, 2025 13:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants