Adding tests for permissions under main use cases #12
Conversation
| self.assertEqual(task_perms["encodeur"], [True, False, True, True, False, True, False, False, False]) | ||
|
|
||
| # change_user(self.portal, "agent") | ||
| # FIXME agent does not have permission to treat the mail |
There was a problem hiding this comment.
@sgeulette Est-ce que tu as une idée de pourquoi l'utilisateur agent n'a pas le droit de "treat" le mail entrant ici ? J'ai déjà regardé avec Thomas et Benoit mais ça n'a rien donné. Tous les champs obligatoires sont bien remplis, un treating est bien assigné et "agent" est bien dans le tg. L'agent devrait avoir le droit "Modify portal Content" sur l'objet mais il n'a aucune permission...
There was a problem hiding this comment.
Le lecteur n'a aussi jamais aucun droit sur quoi que ce soit, ce qui ne m'a pas l'air normal
There was a problem hiding this comment.
@sgeulette up ?
Je n'ai pas non plus fait 3 fichiers comme tu me l'avais conseillé. ça n'est pas encore trop long mais on peut split en plusieurs fichiers facilement si nécessaire
There was a problem hiding this comment.
Je revois ce point dès que c'est plus lisible ;-)
| self.assertFalse(any(task_perms["dirg"])) | ||
| self.assertFalse(any(task_perms["agent"])) | ||
| self.assertFalse(any(task_perms["agent1"])) | ||
| self.assertEqual(task_perms["encodeur"], [True, False, True, True, False, True, False, False, False]) |
There was a problem hiding this comment.
L'écriture des tests comme c'est fait là n'est pas assez claire à la lecture. Il faut avoir en mémoire toutes les permissions pour savoir ce qui est vrai ou faux.
Une possibilité serait de passer en argument à une fonction la liste des permissions (sous une forme d'alias pour faire plus court, ex: 'a_c_i' pour "Access contents information", etc) dans l'idée de faciliter à la lecture la correspondance entre permission et résultat.
| self.pw.doActionFor(imail, "propose_to_manager") | ||
|
|
||
| imail_perms = self.get_perms(imail) | ||
| self.assertFalse(any(imail_perms["chef"])) |
There was a problem hiding this comment.
par défaut, on est dans un workflow sans n+1. Donc chef n'aura normalement jamais de droit.
Une fois que les tests seront bons, il serait intéressant de faire une copie de la méthode de test pour l'appliquer au workflow n+1
| self.assertEqual(task_perms["encodeur"], [True, False, True, True, False, True, False, False, False]) | ||
|
|
||
| # change_user(self.portal, "agent") | ||
| # FIXME agent does not have permission to treat the mail |
There was a problem hiding this comment.
Je revois ce point dès que c'est plus lisible ;-)
| from zope.intid.interfaces import IIntIds | ||
|
|
||
|
|
||
| class TestPermissionsIncomingMail(TestPermissionsBase): |
There was a problem hiding this comment.
@sgeulette J'ai refactoré un peu les tests. Je n'ai fait ue incoming mail pour l'instant. Je ferai les autres si ça te convient comme ça (pour éviter de devoir changer plusieurs fois les emails et les outgoing mails)
| ) | ||
|
|
||
| # change_user(self.portal, "agent") | ||
| # FIXME agent does not have permission to treat the mail |
There was a problem hiding this comment.
Une idée de ceci ?
|
Tu peux revoir les tests en utilisant |
|
Je pense avoir terminé. Les tests passent (178/178).
Dans le ticket DMS-1056, j'ai noté des problèmes de logique potentiels au niveau des permissions. |
chris-adam
changed the title
No description provided.