cmu-delphi/delphi-epidata

Unit tests (even server unit tests) should not make network calls

Aberta

#985 aberto em 28 de set. de 2022

Ver no GitHub
 (0 comentário) (0 reação) (0 responsável)Python (67 forks)auto 404
help wantedtests

Métricas do repositório

Stars
 (104 estrelas)
Métricas de merge de PR
 (Mesclagem média 63d 20h) (11 fundiu PRs em 30d)

Description

The following three tests failed when one of our volunteers was trying to run unit tests without having set up the integration test prerequisites. This shouldn't have been a problem, since the point of unit tests is to check the smallest units of functionality, mocking out more complex functions (like network connections).

It's not immediately obvious why these tests were making network calls -- I suspect that something in _common attempts to connect to the database on import.

We should do one of the following:

  • (ideal) figure out how to properly mock out network calls for flask tests
  • (or if that is not possible) move these tests to the integrations section

Guia do colaborador

Direção de pesquisa
Investigue por que os arquivos de teste mencionados fazem chamadas de rede apesar de serem testes unitários. Trace o caminho de importação em common.py para ver se ele aciona conexões de banco de dados. Em seguida, implemente mocks adequados para testes Flask usando bibliotecas como unittest.mock ou pytest mock.
Pilha de tecnologia
python
Domain
backend
Tipo Issue
Teste
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
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
Pythonpytestmocking
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
65