crytic/slither

Improve test_ast_parsing.py

Open

#681 geöffnet am 10. Nov. 2020

Auf GitHub ansehen
 (0 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)Python (886 Forks)batch import
good first issuehelp wanted

Repository-Metriken

Stars
 (4.769 Stars)
PR-Merge-Metriken
 (Durchschn. Merge 54T 9h) (2 gemergte PRs in 30 T)

Beschreibung

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

Contributor Guide

Research-Richtung
Das Problem liegt in tests/test ast parsing.py. Die aktuellen xfail Markierungen sind nicht versionsspezifisch. Die Lösung besteht darin, einen Mechanismus zu implementieren, der zwei Typen ermöglicht: einen, der für eine bestimmte Version fehlschlägt, und einen, der ab einer Version fehlschlägt. Die in der Issue referenzierte Datei zeigt die Zeilen 81-82. Verknüpfte PRs oder Kommentare bieten keine weiteren Anleitungen.
Tech Stack
python
Domain
testing
Issue Type
Test
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Frisch
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
Pythonpytest
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
70