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] Migration de la route POST api/membership/me/disable (PIX-16733) #11528

Open
wants to merge 4 commits into
base: dev
Choose a base branch
from

Conversation

EmmanuelleBonnemay
Copy link
Contributor

@EmmanuelleBonnemay EmmanuelleBonnemay commented Feb 27, 2025

🥞 Problème

La route POST api/membership/me/disable est encore dans /lib

🥓 Proposition

Migrer cette route

🧃 Remarques

Le test d'acceptance de controller était plutôt un test d'acceptance de route, il a donc été déplacé dans membership.route.test.js

😋 Pour tester

  • Connectez-vous sur Pix Orga avec un compte administrateur (dans une Orga dont vous n'êtes pas le seul administrateur, comme [email protected] pour le college House of the Dragon)
  • Constatez en cliquant sur le sélecteur au dessus du bouton de connexion que [email protected] est membre de 3 organisations, et vérifiez que vous êtes bien sur la page du collège House of the Dragon (sinon choisissez cette organisation)
  • Désactivez vous depuis votre ligne du tableau des membres de l'orga House of the Dragon (onglet "équipe")
  • Constatez que la route POST api/membership/me/disable est appelée avec succès (code http 204) et que vous êtes déconnecté
  • Reconnectez-vous avec le même compte
  • Constatez que [email protected] n'est membre que de deux organisations, et que House of the Dragon n'en fait pas partie.
  • Si vous testez en RA : scalingo -a pix-api-review-pr11528 run npm run db:seed

@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 :

@EmmanuelleBonnemay EmmanuelleBonnemay changed the title Migration de la route POST api/membership/me/disable (PIX-16733) [TECH] Migration de la route POST api/membership/me/disable (PIX-16733) Feb 27, 2025
@P-Jeremy P-Jeremy added this to the Bounded context migration milestone Feb 28, 2025
@P-Jeremy
Copy link
Contributor

Test func OK ✅
Capture d'écran: la route disable est bien appelée avec succès et l'utilisateur et desormais membre de deux orga au lieu de trois

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.

3 participants