facebook/TestSlide

Suggest mock_(async)_callable from StrictMock

Open

#128 aberto em 16 de mar. de 2020

Ver no GitHub
 (0 comments) (0 reactions) (0 assignees)Python (58 forks)auto 404
Better Error MessagesStrictMockenhancementgood first issuehelp wanted

Métricas do repositório

Stars
 (147 stars)
Métricas de merge de PR
 (Métricas PR pendentes)

Description

StrictMock raises when an attribute is accessed, but has not been previously defined

https://github.com/facebookincubator/TestSlide/blob/8d516487247b6591c5a310c3e069ddb32d822d5f/testslide/strict_mock.py#L70

The message simply mentions "setting" the attribute. Let's improve this message to, if the mock has a template, and the attribute is callable, suggest using mock_callable / mock_async_callable to set it.

Guia do colaborador

Direção de pesquisa
Examine a classe StrictMock em testslide/strict mock.py, especificamente o método getattr na linha 70, e modifique a mensagem de erro para verificar se o atributo sendo acessado é chamável e se existe um template, então sugira usar mock callable ou mock async callable.
Pilha de tecnologia
python
Domain
testing qa
Tipo Issue
Refatoração
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
1
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
Menos de 1 hora
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
Python
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
70