cmu-delphi/delphi-epidata

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

Ouverte

#985 ouverte le 28 sept. 2022

Voir sur GitHub
 (0 commentaire) (0 réaction) (0 personne assignée)Python (67 forks)auto 404
help wantedtests

Métriques du dépôt

Stars
 (104 étoiles)
Métriques de merge PR
 (Merge moyen 63j 20h) (11 PRs mergées en 30 j)

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

Guide contributeur

Direction de recherche
Enquêtez sur la raison pour laquelle les fichiers de test mentionnés effectuent des appels réseau malgré le fait qu'ils soient des tests unitaires. Tracez le chemin d'importation dans common.py pour voir s'il déclenche des connexions à la base de données. Ensuite, implémentez des mocks appropriés pour les tests Flask en utilisant des bibliothèques comme unittest.mock ou pytest mock.
Stack technique
python
Domaine
backend
Type d'issue
Test
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
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
Pythonpytestmocking
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
65