Projet

Général

Profil

Development #68039

clean-unused-accounts : lever CommandError en cas d'erreur d'exécution (?)

Ajouté par Frédéric Péters il y a plus d'un an. Mis à jour il y a plus d'un an.

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.

Formats disponibles : Atom PDF