Skip to content

Adds in CPP code coverage and joined JS#499

Merged
tbranyen merged 3 commits into
masterfrom
cpp-code-coverage
Mar 22, 2015
Merged

Adds in CPP code coverage and joined JS#499
tbranyen merged 3 commits into
masterfrom
cpp-code-coverage

Conversation

@tbranyen
Copy link
Copy Markdown
Member

This commit adds in CPP code coverage via GCC and GCOV. That lcov data
is then joined to the JavaScript istanbul output and then generated into
unified test/coverage/report and test/merged.cov reports.

@tbranyen tbranyen force-pushed the cpp-code-coverage branch 3 times, most recently from 79f6e07 to 79f9753 Compare March 20, 2015 04:01
@mattyclarkson
Copy link
Copy Markdown
Collaborator

@tbranyen, this is super awesome. Great work!

@tbranyen
Copy link
Copy Markdown
Member Author

@mattyclarkson ha yea, only issue now is getting Travis to use the right gcov!

Instructions for using this stuff:

export GYP_DEFINES='coverage=1'
npm install
npm test
npm cov
open test/coverage/report/index.html

@coveralls
Copy link
Copy Markdown

Coverage Status

Changes Unknown when pulling 86f38bb on cpp-code-coverage into * on master*.

@tbranyen tbranyen force-pushed the cpp-code-coverage branch 2 times, most recently from 035c01c to 06fda45 Compare March 22, 2015 01:37
This commit adds in CPP code coverage via GCC and GCOV.  That lcov data
is then joined to the JavaScript istanbul output and then generated into
unified test/coverage/report and test/merged.cov reports.
@tbranyen tbranyen force-pushed the cpp-code-coverage branch from 06fda45 to c179e2b Compare March 22, 2015 01:38
tbranyen added a commit that referenced this pull request Mar 22, 2015
Adds in CPP code coverage and joined JS
@tbranyen tbranyen merged commit 3baad61 into master Mar 22, 2015
@tbranyen tbranyen deleted the cpp-code-coverage branch March 22, 2015 01:40
tbranyen added a commit that referenced this pull request Mar 22, 2015
Adds in CPP code coverage and joined JS
johnhaley81 pushed a commit that referenced this pull request Mar 24, 2015
This reverts commit 2b0023b, reversing
changes made to f2b007d.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants