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

Added unit test code coverage report with node-coverage #52

Merged
merged 1 commit into from Jun 1, 2012
Merged

Added unit test code coverage report with node-coverage #52

merged 1 commit into from Jun 1, 2012

Conversation

jleppert
Copy link
Contributor

Added code coverage reporting using node-cover. A report looks like this (CLI, html reports):

http://pastebin.com/u1TLSUbe

@vybs
Copy link
Contributor

vybs commented May 31, 2012

┃ lib/dust.js ┃ 91% ┃ 47 ┃ 533 ┃ 79% ┃ 30 ┃ 147 ┃

@jleppert
Copy link
Contributor Author

jleppert commented Jun 1, 2012

It's not so bad...91% covered :-) This is mainly for making sure we are writing enough and the correct tests for our dust helper additions.

vybs added a commit that referenced this pull request Jun 1, 2012
Added unit test code coverage report with node-coverage
@vybs vybs merged commit 0c000c7 into linkedin:master Jun 1, 2012
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants