Decouple testing of `_conditions.py` from the testing of the public API
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 3/5
- Temps estimé
- 1-2 jours
- Accessibilité débutants
- 45/100
- Type d'issue
- Refactorisation
- Clarté
- Plutôt claire
- Activité
- À l'abandon
- Stack technique
- python
- Domaine
- testing-qa
Piste de recherche
Commencez par lire _conditions.py et tests/test_joined_equal_unequal.py, en particulier le test paramétré autour de la ligne 284. Examinez les autres tests de l’API publique pour vérifier s’ils couvrent des tolérances similaires, puis séparez les vérifications au niveau des conditions du comportement spécifique à l’API. Le travail est terminé lorsque les tests de l’API publique ne dupliquent plus les tests pris en charge par _conditions.py, tout en conservant la couverture.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
From @EgeKaraismailogluQC:
We are currenly conflating the testing of some functionality defined in _conditions.py with the testing of the public API. For example this test which is meant to test the correctness of joined_equal_unequal is parameterized to test that comparisons with relative/absolute tolerance work as expected.
It would be cleaner if this test, and other tests for the public API assume that the methods defined in _conditions.py are implemented correctly and instead focused on testing only the functionality related to their respective API methods. This way, we can get rid of duplicate testing.
- Langage dominant
- Python
- Étoiles
- 47
- Forks
- 1
- Merge moyen
- 1 j 7 h
- PR mergées (30 j)
- 8
Préparer son environnement
Nous n'avons pas encore vérifié les fichiers d'installation de ce projet. Commencez par son README, et consultez notre guide de la première contribution pour les étapes générales.
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.
Autres issues de Quantco/diffly
-
Difficulté 4/5 3-5 jours Accessibilité débutants 35/100
-
Refactor test setupOuverte
Difficulté 5/5 Plus d'une semaine Accessibilité débutants 20/100
-
Difficulté 5/5 Plus d'une semaine Accessibilité débutants 35/100
-
Difficulté 5/5 Plus d'une semaine Accessibilité débutants 25/100
Toutes les issues de Quantco/diffly
Issues similaires
-
ACK_WAITING HELP_WANTED UPDATE_CS
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
OWASP/CheatSheetSeries#2458 ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 82/100
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 90/100
BasedHardware/omi#19711 ·
Les mainteneurs répondent en général sous 1 jour
-
Qwen3_5MoeModel no longer returns router_logits, breaking aux loss with output_router_logits=TrueOuverte
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
huggingface/transformers#49172 ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 82/100
vllm-project/vllm-metal#885 ·
Les mainteneurs répondent en général sous 1 jour