search

nodejs / llnode

An lldb plugin for Node.js and V8, which enables inspection of JavaScript states for insights into Node.js processes and their core dumps.
Other
1.15k stars 100 forks source link

Add code coverage to CI #258

Closed mmarchini closed 5 years ago

mmarchini commented 5 years ago

Add support to gcov through a npm install flag (--llnode_coverage=true), and update Travis settings to upload coverage report to CodeCov on success.

codecov-io commented 5 years ago

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@59b65c4). Click here to learn what that means. The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##             master     #258   +/-   ##
=========================================
  Coverage          ?   72.09%           
=========================================
  Files             ?       34           
  Lines             ?     4279           
  Branches          ?        0           
=========================================
  Hits              ?     3085           
  Misses            ?     1194           
  Partials          ?        0

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 59b65c4...acd5f56. Read the comment docs.

mmarchini commented 5 years ago

Oh cool, seems like we don't need to set up anything in the repository 😄

mmarchini commented 5 years ago

I added code coverage report for our tests as well (with nyc):

image

mmarchini commented 5 years ago

cc @joyeecheung @cjihrig

mmarchini commented 5 years ago

Landed in 59b65c4...9f3dce3