PyCQA/flake8-bugbear

Report on uncatchable exception handlers (?)

Aberta

#349 aberto em 7 de fev. de 2023

Ver no GitHub
 (3 comentários) (0 reação) (0 responsável)Python (116 forks)github user discovery
enhancementhelp wanted

Métricas do repositório

Stars
 (1.116 estrelas)
Métricas de merge de PR
 (Mesclagem média 4h 17m) (3 fundiu PRs em 30d)

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.

Guia do colaborador

Direção de pesquisa
Estude a estrutura do plugin flake8 bugbear, focando em como as verificações existentes, como B001, são implementadas. Observe a representação AST dos blocos try/except usando o módulo ast do Python. Identifique as classes de exceção embutidas e sua hierarquia. Escreva uma nova verificação que itera sobre os manipuladores except em um bloco try e usa issubclass para determinar se um tipo de exceção é uma subclasse de qualquer exceção capturada anteriormente (excluindo BaseException para bare except). Considere casos extremos como vinculação de variáveis (except Exception as e) e tuplas de exceções. Adicione um caso de teste e registre a verificação.
Pilha de tecnologia
python
Domain
devtools
Tipo Issue
Funcionalidade
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
3
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
Pythonflake8
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
65