crytic/slither

Improve test_ast_parsing.py

Open

#681 ouverte le 10 nov. 2020

Voir sur GitHub
 (0 commentaires) (0 réactions) (0 assignés)Python (886 forks)batch import
good first issuehelp wanted

Métriques du dépôt

Stars
 (4 769 stars)
Métriques de merge PR
 (Merge moyen 54j 9h) (2 PRs mergées en 30 j)

Description

One issue with our current parsing pytest is that it is not robust against new version of solc for the xfail tests. For example, these tests are not working with 0.7.1 and 0.7.2: https://github.com/crytic/slither/blob/ae7c410938b616d993e6c27678f6e48d9a4d7dd6/tests/test_ast_parsing.py#L81-L82

And was not updated when 0.7.3/0.7.4 were released. So the tests fail, while the script should have expected to not support 0.7.3 and 0.7.4 for this test.

We should have two version of XFAIL: the ones that fail for a specific version, and the ones that fail from a version

Guide contributeur

Direction de recherche
Le problème se trouve dans tests/test ast parsing.py. Les marqueurs xfail actuels ne sont pas spécifiques à une version. La solution est de mettre en œuvre un mécanisme pour avoir deux types : un qui échoue pour une version spécifique et un qui échoue à partir d'une version donnée. Le fichier référencé dans l'issue montre les lignes 81-82. Aucune PR ou commentaire lié ne fournit de conseils supplémentaires.
Stack technique
python
Domaine
testing
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.
Récente
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