search

hedyhli / starcli

:sparkles: Browse trending GitHub projects from your command line
https://pypi.org/project/starcli/
MIT License
555 stars 50 forks source link

click cli testing #77

Closed jSadoski closed 4 years ago

jSadoski commented 4 years ago

Context

closes #41

Description

This adds a test suite using click.testing for cli tests. It also adds a feature to allow pytest to use an --auth option in the same way it's used with starcli. This allows the tests to run much faster and test the functionality of click with valid API credentials.

I'm sure these tests could be improved; I hope this serves as a starting point to write better tests.

Known issue: Right now, tests/test_search is not using authentication, and if all the tests are run at once, the rate limit causes those tests to fail. tests/test_search still works if you run the file independently (python -m pytest tests/test_search.py). It should be easy to fix soon.

jSadoski commented 4 years ago

Looks like I'll have to work on test_search.py now!