Project

General

Profile

Development #68039

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

Added by Frédéric Péters 6 months ago. Updated 6 months ago.

Status:
Nouveau
Priority:
Normal
Assignee:
-
Category:
-
Target version:
-
Start date:
09 August 2022
Due date:
% Done:

0%

Estimated time:
Patch proposed:
No
Planning:
No

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.

Also available in: Atom PDF