Development #68039
clean-unused-accounts : lever CommandError en cas d'erreur d'exécution (?)
Statut:
Nouveau
Priorité:
Normal
Assigné à:
-
Catégorie:
-
Version cible:
-
Début:
09 août 2022
Echéance:
% réalisé:
0%
Temps estimé:
Patch proposed:
Non
Planning:
Non
Description
Depuis #41284 la commande se termine ainsi
try: self.clean_unused_accounts() except Exception: logger.exception('clean-unused-accounts failed')
Pour l'utilisation qui s'en fait actuellement, appelée depuis uwsgi, ça ne change pas grand chose mais dans les tests, ça m'a surpris que malgré une erreur call_command('clean-unused-accounts')
passe avec succès.
Dans #67998 j'ai mis assert 'clean-unused-accounts failed' not in caplog.text
pour détecter l'erreur mais je ne trouve pas ça terrible. Je proposerais plutôt qu'une exception CommandError soit levée.