Development #14723
clarifier les traces reçues lors de hobo_notify
0%
Description
Il y a des "assert" peu verbeuses dans hobo_notify.py qui font des traces du genre :
Exception: type = '<type 'exceptions.AssertionError'>', value = '' Stack trace (most recent call first): File "/usr/lib/python2.7/dist-packages/wcs/ctl/hobo_notify.py", line 215, in provision_user 213 user.remove_self() 214 except Exception, e: > 215 publisher.notify_of_exception(sys.exc_info(), context='[PROVISIONNING]') 216 217 CmdHoboNotify.register()
Je propose de les remplacer par des if/raise avec des exceptions plus explicites.
Fichiers
Révisions associées
Historique
Mis à jour par Thomas Noël il y a plus de 7 ans
- Fichier 0001-hobo_notify-raise-more-explicit-exceptions-14723.patch 0001-hobo_notify-raise-more-explicit-exceptions-14723.patch ajouté
- Statut changé de Nouveau à En cours
- Patch proposed changé de Non à Oui
Mis à jour par Frédéric Péters il y a plus de 7 ans
On peut taper un commentaire à côté de l'assert, genre assert 'uuid' in o, 'role without uuid' et ça passera le message dans l'exception.
Mis à jour par Thomas Noël il y a plus de 7 ans
Frédéric Péters a écrit :
On peut taper un commentaire à côté de l'assert
Comme assert nécessite "__debug__" pour fonctionner, je n'aime pas trop l'utiliser dans du code (ok, on fait toujours tourner nos codes avec debug, mais quand même).
Mis à jour par Frédéric Péters il y a plus de 7 ans
Véritable préoccupation, à fonctionner avec des conditions, on baisse le pourcentage de couverture de code.
- assert 'uuid' in o
+ if 'uuid' not in o:
+ raise KeyError('role without uuid')
uuid = o['uuid'].encode(publisher.site_charset)
En vrai aussi, sur celle-ci (et l'autre identique pour les utilisateurs), le KeyError on se le taperait de manière native, sans rien faire.
Mis à jour par Thomas Noël il y a plus de 7 ans
- Fichier 0001-hobo_notify-raise-more-explicit-exceptions-14723.patch 0001-hobo_notify-raise-more-explicit-exceptions-14723.patch ajouté
Quand lama faché, lui toujours perdre du temps comme ça.
Mis à jour par Thomas Noël il y a plus de 7 ans
- Statut changé de En cours à Résolu (à déployer)
commit 1f06e4187ce13875bc25b8c0d4fb85835be37d4b
Author: Thomas NOEL <tnoel@entrouvert.com>
Date: Sat Jan 21 16:05:03 2017 +0100
hobo_notify: raise more explicit exceptions (#14723)
Mis à jour par Frédéric Péters il y a plus de 5 ans
- Statut changé de Résolu (à déployer) à Solution déployée
hobo_notify: raise more explicit exceptions (#14723)