aio-libs/multidict

[TODO] Replace `tests/gen_pickles.py` with a pytest plugin

Ouverte

#922 ouverte le 16 janv. 2024

Voir sur GitHubGuide contributeur
 (13 commentaires) (2 réactions) (1 personne assignée)Python (130 forks)auto 404
Hacktoberfestenhancementgood first issuehelp wanted

Métriques du dépôt

Stars
 (491 étoiles)
Métriques de merge PR
 (Métriques PR en attente)

Description

The idea is simple:

  • Make a session-scoped fixture in tests/conftest.py
  • Move the logic from gen_pickles.py there and generate the pickles in a tmppath-provided dir
  • Integrate that into the pickling tests
  • Delete tests/gen_pickles.py and the pickle files in the tests/ dir
  • Bonus points: keep pickles in-memory, so they don't hit the disk
  • Bonus 2: check if a session fixture is needed and switch to the normal function scope, if not

Guide contributeur

Direction de recherche
Lisez `tests/gen pickles.py` et comprenez la logique de génération des pickles. Apprenez les fixtures pytest, en particulier celles de portée session et conftest.py. Ensuite, déplacez la logique dans une fixture dans conftest.py, intégrez la aux tests existants et supprimez enfin l'ancien script et les fichiers pickle.
Stack technique
python
Domaine
testing
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-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
Pythonpytest
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
75