[FEATURE] Modifier l’API permettant d’accepter les CGU afin de fonctionner avec le nouveau modèle (PIX-15588)

[lego-technix]

🎄 Problème

Afin de mettre en place le versionement des CGU de Pix Orga, il est nécessaire de modifier l’API permettant d’accepter les CGU afin de fonctionner avec le nouveau modèle.

🎁 Proposition

Pour la route PATCH /api/users/{id}/pix-orga-terms-of-service-acceptance, modifier le use case api/src/identity-access-management/domain/usecases/accept-pix-orga-terms-of-service.usecase.js pour modifier les informations de CGU en se basant sur la nouvelle API acceptLegalDocumentByUserId du contexte legal-document.

Pour simplifier le code on propose également de changer la valeur et le status code HTTP de retour de userController.acceptPixOrgaTermsOfService pour qu'il n'y ait pas de valeur de retour et que le status code HTTP soit 204. Cela est compatible avec la manière dont le code Ember du Front Pix Orga est réalisé, mais c'est un point qu'il faudra bien vérifier avec des tests de non-régression.

🧦 Remarques

• ⚠️ Le usecase api/src/legal-documents/domain/usecases/accept-legal-document-by-user-id.usecase.js ne tire pas parti du feature flag isLegalDocumentsVersioningEnabled comme le fait le usecase api/src/legal-documents/domain/usecases/get-legal-document-status-by-user-id.usecase.js. Mais il nous semble que tirer parti du feature flag aussi bien à la récupération qu'à la modification assurerait la non-interférence complète du nouveau code en production. Ce qui correspond à ce qu'on attend de l'utilisation d'un tel feature flag. Cette PR ne contient pas encore cette modification car cela n'avait pas été décidé ainsi au craquage. Point à discuter donc avant de finaliser cette PR.
• Un dernier commit a été ajouté pour rendre les usecases du context « legal documents » transactionnels.

🎅 Pour tester

Tester d'abord avec des CGU non-existantes dans le nouveau modèle

1. Faire une invitation à Pix Orga (par exemple depuis Pix Admin avec le compte [email protected]) à un utilisateur non-existant (par exemple [email protected])
2. Accepter cette invitation en créant un nouveau compte (Je m’inscris) et en acceptant les CGU
3. Vérifier que la route /api/users/{id}/pix-orga-terms-of-service-acceptance est appelée sans erreur
4. Vérifier que la table users a bien été modifiée et que le nouvel utilisateur a été créé avec les valeurs suivantes :
   • pixOrgaTermsOfServiceAccepted true
   • lastPixOrgaTermsOfServiceValidatedAt avec la bonne date de création

select "id", "email", "username", "createdAt", "updatedAt", "pixOrgaTermsOfServiceAccepted", "lastPixOrgaTermsOfServiceValidatedAt" from "users" order by "updatedAt" desc limit 1;

Tester ensuite avec des CGU existantes dans le nouveau modèle

1. Créer au préalable les CGU de Pix Orga avec le nouveau modèle avec la commande suivante :

node src/legal-documents/scripts/add-new-legal-document-version.js --type 'TOS' --service 'pix-orga' --versionAt '2024-01-02'

2. Faire une invitation à Pix Orga (par exemple depuis Pix Admin avec le compte [email protected]) à un utilisateur non-existant (par exemple [email protected])
3. Accepter cette invitation en créant un nouveau compte (Je m’inscris) et en acceptant les CGU
4. Vérifier que la route /api/users/{id}/pix-orga-terms-of-service-acceptance est appelée sans erreur
5. Vérifier que la table users et aussi la table legal-document-version-user-acceptances ont bien été modifiées.

Vérifier que le nouvel utilisateur a été avec les valeurs suivantes :

• pixOrgaTermsOfServiceAccepted true
• lastPixOrgaTermsOfServiceValidatedAt avec la bonne date de création

select "id", "email", "username", "createdAt", "updatedAt", "pixOrgaTermsOfServiceAccepted", "lastPixOrgaTermsOfServiceValidatedAt" from "users" order by "updatedAt" desc limit 1;

Vérifier que l'acceptation des CGU a bien été enregistrée pour cet utilisateur avec l'acceptation avec la bonne valeur acceptedAt :

select * from "legal-document-version-user-acceptances";

[pix-bot-github]

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

• App (.fr)
• App (.org)
• Orga (.fr)
• Orga (.org)
• Certif (.fr)
• Certif (.org)
• Junior
• Admin
• API
• Audit Logger

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

• scalingo front
• scalingo api
• scalingo audit-logger

[EmmanuelleBonnemay]

Lu et testé avec succès