PyCQA/flake8-bugbear

Report on uncatchable exception handlers (?)

Ouverte

#349 ouverte le 7 févr. 2023

Voir sur GitHub
 (3 commentaires) (0 réaction) (0 personne assignée)Python (116 forks)github user discovery
enhancementhelp wanted

Métriques du dépôt

Stars
 (1 116 étoiles)
Métriques de merge PR
 (Merge moyen 4h 17m) (3 PRs mergées en 30 j)

Description

Was surprised bugbear didn't report this, but not encountered in live code so I don't know if it's seen much / ever in the wild:

try:
  ...
except BaseException:
  ...
except:
  ...

will report B001/E722, but in a project that uses bare excepts widely and disables those checks the except never being possible to run seems like a separate error.

try:
  ...
except BaseException:
  ...
except Exception:
  ...

Gives no warning, nor does a plausibly more common

try:
  ...
except Exception:
  ...
except ValueError:
  ...

or similar variants. Checking whether a caught exception is a built-in, and if so issubclass of a previously caught exception, seems straightforward. I quickly checked mypy and vulture as well and neither of them sees it. But this might definitely not be common enough that it's worth bothering with.

Guide contributeur

Direction de recherche
Étudiez la structure du plugin flake8 bugbear, en vous concentrant sur la manière dont les vérifications existantes comme B001 sont implémentées. Examinez la représentation AST des blocs try/except à l'aide du module ast de Python. Identifiez les classes d'exceptions intégrées et leur hiérarchie. Écrivez une nouvelle vérification qui itère sur les gestionnaires except dans un bloc try et utilise issubclass pour déterminer si un type d'exception est une sous classe de toute exception précédemment attrapée (en excluant BaseException pour bare except). Considérez les cas limites comme la liaison de variables (except Exception as e) et les tuples d'exceptions. Ajoutez un cas de test et enregistrez la vérification.
Stack technique
python
Domaine
devtools
Type d'issue
Fonctionnalité
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
Pythonflake8
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
65