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

Test Coverage #63

Open
pryrt opened this issue Jan 31, 2021 · 1 comment
Open

Test Coverage #63

pryrt opened this issue Jan 31, 2021 · 1 comment
Assignees
Labels
nice-v1.0 would be nice to have by v1.0 test suite

Comments

@pryrt
Copy link
Owner

pryrt commented Jan 31, 2021

Coverage has fallen way out-of-date; there are significant error-handling sections that should probably be added -- even if I don't get to 100%, I should go through the coverage holes and see if there are easy fixes before releasing v0.006.

@pryrt pryrt added test suite nice-v1.0 would be nice to have by v1.0 labels Jan 31, 2021
@pryrt pryrt self-assigned this Jan 31, 2021
@pryrt pryrt changed the title v0.006 Coverage Test Coverage Mar 1, 2021
@pryrt
Copy link
Owner Author

pryrt commented Mar 1, 2021

I've made drastic improvements in v0.006, but there are a lot of conditions and branches that aren't tested; I'm finding most are x//''-type in debugging statements (which are useful when I'm debugging changes, not for the end-user) and other defensive-coding choices.

Most importantly, all subroutines are tested (except maybe a mysterious sub in Editor.pm which it won't identify in the sub-coverage page, despite saying 98.1%)

I'll leave this one open (renamed without v0.006 in the name), but getting to 100% coverage won't be doable in a reasonable amount of time, and I want to release a version that handles NPP v7.9.1 and newer

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
nice-v1.0 would be nice to have by v1.0 test suite
Projects
None yet
Development

No branches or pull requests

1 participant