next-exp/IC

Reorganize, simplify and clean up fixtures

Open

#913 ouverte le 30 oct. 2024

Voir sur GitHub
 (0 commentaires) (0 réactions) (1 assigné)Python (77 forks)auto 404
easygood first issuehelp wantedtesting

Métriques du dépôt

Stars
 (10 stars)
Métriques de merge PR
 (Métriques PR en attente)

Description

We use many fixtures. Most of them are thrown into invisible_cities/conftest.py without thinking much. Let's reorganize them following the following logic:

  • If they are used only in one file, move them to that file
  • If they are used only in one folder, move them to the conftest.py file in that folder
  • Otherwise, keep them in invisible_cities/conftest.py

Moreover, some of those fixtures are monstrous. Let's try to simplify them or replace them with test files whenever possible to keep things readable.

Guide contributeur

Direction de recherche
Identifiez les fixtures utilisées uniquement dans un fichier et déplacez les dans ce fichier ; identifiez les fixtures utilisées uniquement dans un dossier et déplacez les dans le conftest.py de ce dossier ; sinon, conservez les dans le conftest.py racine. Pour les fixtures monstrueuses, envisagez de les simplifier ou de les remplacer par des fichiers de test.
Stack technique
python
Domaine
testing qa
Type d'issue
Refactorisation
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-2 jours
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
Pythonpytest
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
70