Skip to content
This repository has been archived by the owner on Jun 24, 2020. It is now read-only.

Generate code coverage reports #329

Merged
merged 2 commits into from
Feb 6, 2015
Merged

Generate code coverage reports #329

merged 2 commits into from
Feb 6, 2015

Conversation

contolini
Copy link
Member

This replaces the mochaTest grunt task with mocha_istanbul to track code coverage. Coverage reports are output to test/coverage and sent to coveralls.io. A coverage badge has been added to the README.

@ascott1
Copy link
Member

ascott1 commented Feb 6, 2015

ohhhhh, this is exciting

ascott1 added a commit that referenced this pull request Feb 6, 2015
Generate code coverage reports
@ascott1 ascott1 merged commit 99d9a59 into cfpb:master Feb 6, 2015
@mthibos
Copy link
Contributor

mthibos commented Feb 6, 2015

@ascott1 What's a code coverage report? I'm excited too, just want to know what about! 😀

@stephanieosan
Copy link
Member

Test coverage?

@contolini
Copy link
Member Author

@stephanieosan Bingo. It's called code coverage instead of test coverage because you're covering the code in tests.

It's good practice to write automated tests that verify your code works as expected (and that it doesn't break as updates are made down the road). As a codebase grows, it can be hard to keep track of which parts of the application are covered by tests and which are not.

Code coverage tools tell you how much of your code is covered by tests. They generate reports that tell you "Hey! Function ABC and XYZ don't have any tests written for them." They'll also tell you the percentage of your code that is covered by tests. e.g. If you update the application and your code coverage drops from 100% to 85%, it means you failed to write proper tests for that new code.

@stephanieosan
Copy link
Member

I love this!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants