kunklejr / node-covershot Goto Github PK

For projects that use rewire, it's difficult to replace require() calls with csrequire(). Could you add support for rewire?

Followed the steps as on the homepage (apart from jsmeter which will not work)

Resulted in error:

Error: No data to merge
at module.exports (/Users/d.budge/Sites/node-app/test/node_modules/covershot/lib/merge.js:81:11)

I'm using covershot successfully with mocha and TJ's version of node-jscoverage with this simple script.

I did require my fix for printing source lines in #9 before I got source in the HTML report.

Would be great to get that into the docs so people aren't scared off by having to rewrite all the require()s in their tests. :)

It appear that the format informations used to create the LCOV output don't take in account the full path of the covered file.

Makes it harder to analyze further if using nested directories.

The link to jsmeter2 (https://github.com/joeferner/node-jsmeter) is no longer valid.

It would be great if covershot was compatible with Jenkins.

Services like coveralls.io support only lcov format. Would be nice to be able to use it seamlessly

Maybe there's a way to include the whole path in the report (perhaps as a flag)

I'd like to covershot several directories. From reading the covershot docs, I suppose I should do:

var covershot = require('covershot');

// replace all paths containing a 'routes' component with 'routes-cov'
covershot.replace('routes', 'routes-cov');
covershot.replace('services', 'services-cov');
covershot.replace('email', 'email-cov');

var csrequire = covershot.require.bind(null, require);
var myLibrary = csrequire('../lib/myLibrary');

But when I do this, csrequire doesn't appear to actually be replacing these, and so Node reports errors about failing to find files.