Notifications: Continuer à envoyer les notifications aux anciens collaborateurs

[tonial]

For now.

🤔 Pourquoi ?

Indiquez le problème que nous sommes en train de résoudre et les objectifs métiers ou techniques qui sont visés par ces changements.

🍰 Comment ?

Décrivez en quelques mots la solution retenue et mise en oeuvre, les difficultés ou problèmes rencontrés. Attirez l'attention sur les décisions d'architecture ou de conception importantes.

🚨 À vérifier

• Mettre à jour le CHANGELOG_breaking_changes.md ?

🏝️ Comment tester

Les instructions pour reproduire le problème, les profils de test, le parcours spécifique à utiliser, etc. Si vous disposez d'une recette jetable, mettre l'URL pour tester dans cette partie.

💻 Captures d'écran

[xavfernandez]

Suggested change

	assert set(sum((mail.to for mail in mailoutbox), [])) == {admin_1.email, admin_2.email, self.user.email}
	assert set(*(mail.to for mail in mailoutbox)) == {admin_1.email, admin_2.email, self.user.email}

[tonial]

Je vois pas comment ça pourrait marcher vu que mail.to est une liste.
A priori il n'y a qu'un seul élément dedans, mais c'est mieux de le vérifier

Sinon on peut faire assert set(mail.to[0] for mail in mailoutbox) == {admin_1.email, admin_2.email, user.email}

[xavfernandez]

arf effectivement 🤦

Tu as aussi set(itertools.chain.from_iterable(mail.to for mail in mailoutbox)) en alternative 😛

[tonial]

Ah, ça c'est cool, je connaissais pas !

[xavfernandez]

Suggested change

	assert set(sum((mail.to for mail in mailoutbox), [])) == {admin_1.email, admin_2.email, user.email}
	assert set(*(mail.to for mail in mailoutbox)) == {admin_1.email, admin_2.email, user.email}

[tonial]

Je merge pour rétablir le bon fonctionnement, mais la discussion sur ta syntaxe alternative n'est pas fermée : je passe peut être à côté d'un truc cool de python alors ça m'intéresse de savoir ce qu'il en est :)