zestedesavoir/zds-site — pull request #1082 by poulp (2014-07-07 17:50:00+00:00)
Title: Ajout de tests pour les mp et correction de bugs

| Q | R
| ------------------------- | -------------------------------------------
| Correction de bugs ? | oui
| Nouvelle Fonctionnalité ? | non
| Tickets concernés | -

Même architecture de tests que les galeries, quelques remarques (corrigées):

* Dans les modèles : l'utilisation de la fonction `get_current_user` pose problème dans les tests, du coup je l'ai mise en paramètre optionel dans les méthodes concernées (pour pas poser de problème dans le code en général). Mais j'aimerais bien discuter de cette fonction, car je ne vois pas ce qu'elle apporte par rapport à un `request.user`.
* Petit correction dans la méthode `first_unread_post`, un attribut qui n'existe pas ça fait pas du bien ;)
* Il manque des vérifications de droits dans les contrôleurs :
* Suppression de n'importe quel mp
* On peut s'ajouter à une conversation privée
* Pas de vérification sur les droits de réponses (bon la on tombe sur une erreur 500)
* Un utilisateur peut s'envoyer un mp tout seul

Voila voila :)