Suppression de compte #57 #74
No reviewers
felix
Labels
No labels
bug
documentation
fonctionnalité
No milestone
No project
No assignees
2 participants
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
yaal/escut!74
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "delete-account"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Résoud #57
Un utilisateur peut supprimer son compte lorsqu'il n'a plus d'abonnement actif. Le compte Canaille est supprimé via SCIM, le compte Lago est supprimé et les factures non réglées sont toujours dans le système pour être suivies.
S'il n'y a aucune facture non réglée, le compte Stripe est aussi supprimé.
S'il y a une ou plusieurs factures non réglées, le compte Stripe n'est pas supprimé, il le sera automatiquement lorsque les paiements de toutes les factures seront marqués "successful" (ou que les factures seront void).
@ -82,0 +104,4 @@with patch("stripe.Customer") as stripe_customer:stripe_customer.return_value = {"data": [stripe_user]}res = client.get("/suppression")assert ("success", "Votre compte a bien été supprimé.") in res.flashesthis test should also verify that the canaille user is correctly deleted, and that the information email is correctly sent
c'est testé ✅
yaal/escut@11dce1f85b@ -61,0 +73,4 @@)if user:try:user = scim.delete(User, username)I think this function takes
user.idinstead of the usernameDans Canaille l'id d'un utilisateur c'est son
username, donc ça fonctionne avecusername, mais je peux remplacerusernameici paruser_idpour que ça suive ce que demande SCIM et toujours lui passer l'username (leexternal_iden l'occurrence) quand la fonction est appelée et ça fonctionnera aussi. Ca te parait ok ?j'ai fait ça du coup
yaal/escut@d3fd8b48c8il y a encore le pb des autres tests qui fail à cause du mock de Lago à résoudre avant de merge
on utilise des uuid dans le tests plutôt que des username écrits en dur.
Le test
test_fail_login_callback_cannot_retrieve_user_from_lagofail occasionnellement à cause du parallélisme (l'utilisateur canaille est supprimé dans les tests de suppression de compte avant que ce test obtienne une autorisation de canaille, donc canaille ne retrouve pas l'utilisateur et retourne une erreur prématurée)2d126156cbd3fd8b48c8