Split up _testcapimodule.c
Personne n'a encore pris cette issue.
- Langage dominant
- Python
- Étoiles
- 77.2k
- Forks
- 36k
- Merge moyen
- 1 j 9 h
- PR mergées (30 j)
- 558
Description
Modules/_testcapimodule.c is a nearly-8000-line behemoth with no clear structure or organization. It is getting hard to maintain.
It also doesn't work well with testing (a) feature macros that affect Python.h and (b) module initialization, so we have additional C-API testing modules: _testmultiphase, _testimportmultiple, _testinternalcapi.
_testbuffer is already split out, but there are many other aspects of the API that would use a similar dedicated test suite.
We should split and combine these, ideally without polluting the namespace of top-level modules.
- PR: gh-94549
- PR: gh-94695
- PR: gh-94702
- PR: gh-95386
- PR: gh-95793
- PR: gh-95819
- PR: gh-99346
- PR: gh-99494
- PR: gh-99517
- PR: gh-99532
- PR: gh-99549
- PR: gh-99551
- PR: gh-102173
- PR: gh-104403
- PR: gh-129400
- PR: gh-129512
- PR: gh-129516
- PR: gh-129521
- PR: gh-129544
Guide de contribution
Ouvrir le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Piste de recherche
Commencez par examiner Modules/_testcapimodule.c et les modules existants _testmultiphase, _testimportmultiple, _testinternalcapi et _testbuffer, puis examinez les PR liés indiqués dans l’issue. Le travail est considéré comme terminé lorsque les tests de la C-API sont répartis dans des modules cohérents, que la couverture de tests pertinente est préservée et que la pollution de l’espace de noms des modules de niveau supérieur est évitée.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Évaluation
- Stack technique
- c, python
- Domaine
- testing-qa
- Type d'issue
- Refactorisation
- Difficulté
- 5/5
- Temps estimé
- Plus d'une semaine
- Activité
- À l'abandon
- Clarté
- Plutôt claire
- Accessibilité débutants
- 20/100