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.

Sign up for GitHub

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

Jump to bottom

Improve test coverage #42

Closed
Shadow53 opened this issue Oct 3, 2021 · 0 comments · Fixed by #43
Closed

Improve test coverage #42

Shadow53 opened this issue Oct 3, 2021 · 0 comments · Fixed by #43
Assignees
@Shadow53
Labels
tests Tests are missing or failing
Milestone
0.3.0

Comments

@Shadow53
Copy link
Owner

Shadow53 commented Oct 3, 2021

#40 just got fixed, which greatly improved calculated code coverage by including the Python-based integration tests. I'd like to further improve code coverage, with a goal of >90% coverage, 95% if possible.
@Shadow53 Shadow53 added the tests Tests are missing or failing label Oct 3, 2021
@Shadow53 Shadow53 added this to the 0.3.0 milestone Oct 3, 2021
@Shadow53 Shadow53 self-assigned this Oct 3, 2021
@Shadow53 Shadow53 mentioned this issue Oct 8, 2021
Merged
Shadow53 added a commit that referenced this issue Oct 8, 2021
@Shadow53
Implement better test coverage (#42) (#43)
a9db936 
* fix: improve test coverage

Most of the improvements in this commit come from ignoring lines that
are not reasonable to test and/or do not contain logic worth checking
the coverage of. See `CONTRIBUTING.md` for more.

* test: increase test coverage

* chore: fix Makefile.toml for closing brace

* ci: apply coverage exclusion rules to CI

* style: remove unnecessary comments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Shadow53 Shadow53

Labels
tests Tests are missing or failing
Projects
None yet
Milestone
0.3.0
Development

Successfully merging a pull request may close this issue.

Implement better test coverage (#42) Shadow53/hoard
1 participant
@Shadow53