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

Check test coverage and increase/fix where necessary #3

Closed
leejo opened this issue May 18, 2014 · 1 comment
Closed

Check test coverage and increase/fix where necessary #3

leejo opened this issue May 18, 2014 · 1 comment

Comments

@leejo
Copy link
Owner

leejo commented May 18, 2014

Check existing test coverage with Devel::Cover, et al, and increase where necessary. I would like to make sure we have a good level of test coverage, this can be done as a rolling task when addressing other issues.

@leejo
Copy link
Owner Author

leejo commented Dec 1, 2014

4364c01 added coveralls hook in travis CI, which is currently reporting 87%. I believe this figure is wrong due to CGI.pm's autoloading optimisation that string evals most of the module's subroutines - this is a compile time optimisation that needs to die. Profile compilation time with and without this optimisation to get better coverage metrics and thus a better understanding of what needs to happen to increase test coverage.

@leejo leejo closed this as completed in 43d401c Apr 17, 2015
leejo added a commit that referenced this issue Apr 17, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant