Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Increase Test Coverage #206

Open
allerter opened this issue Apr 24, 2021 · 0 comments
Open

Increase Test Coverage #206

allerter opened this issue Apr 24, 2021 · 0 comments

Comments

@allerter
Copy link
Collaborator

allerter commented Apr 24, 2021

I tested the coverage of the unit tests using coveragepy, and the coverage stands at about %75. The HTML report is available here.
After releasing LyricsGenius 3.0.0, there were some issues that we could've easily avoided if we had more tests (#181, #182, #183, #184) . Achieving a coverage of %95 or more will help prevent these issues going forward. But before doing so, we should migrate the tests to pytest to make it easier to add more tests (#205).

To reproduce the report:

git clone https://github.com/allerter/LyricsGenius.git
cd LyricsGenius
git checkout test-coverage
pip install -e .[dev]
tox -e coverage
coverage html
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant