BlueBrain / BlueBrain/Atlas-Download-Tools
Clean up pytest markers
Open
- Dominant language
- Python
- Stars
- 17
- Forks
- 8
- PR merge metrics
- No merged PRs in 30d
Description
https://github.com/BlueBrain/Atlas-Download-Tools/blob/829206f9a2a21720bebc8830d51971315fccde27/tox.ini#L70-L83
It seems like the `todo` and `slow` markers are not used in the code anymore. Do you think we should keep them? Additionally, there are some tests that requires internet and are not marked with the marker `internet`. See below an example (maybe there are more?).
https://github.com/BlueBrain/Atlas-Download-Tools/blob/829206f9a2a21720bebc8830d51971315fccde27/tests/test_utils.py#L521-L529
Anyway, I am not sure how useful the `internet` marker is, however, it can help us identify and filter tests that require it.
Contributor guide
Assessment
This issue has not been assessed yet.