crytic/slither

Improve test_ast_parsing.py

Open

#681 aperta il 10 nov 2020

Vedi su GitHub
 (0 commenti) (0 reazioni) (0 assegnatari)Python (886 fork)batch import
good first issuehelp wanted

Metriche repository

Star
 (4769 star)
Metriche merge PR
 (Merge medio 54g 9h) (2 PR mergiate in 30 g)

Descrizione

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

Guida contributor

Direzione di ricerca
Il problema si trova in tests/test ast parsing.py. Gli attuali marcatori xfail non sono specifici per versione. La soluzione è implementare un meccanismo per avere due tipi: uno che fallisce per una versione specifica e uno che fallisce da una versione in poi. Il file a cui si fa riferimento nell'issue mostra le righe 81-82. Nessun PR o commento collegato fornisce ulteriori indicazioni.
Tech stack
python
Dominio
testing
Tipo issue
Test
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Nuova
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
Pythonpytest
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70