Add coverage script - Githubissues

nodejs / webcrypto

This repository has been archived. The WebCrypto API has been implemented in recent versions of Node.js and does not require additional packages.

69 stars 20 forks source link

Add coverage script #9

Closed tniessen closed 5 years ago

tniessen commented 5 years ago

Add coverage using Istanbul.

panva commented 5 years ago

Have you considered c8 (uses the native NODE_V8_COVERAGE)?

Especially for a node project i think it's a nice way of applying Eat Your Own Dog Food.

tniessen commented 5 years ago

I don't have a strong opinion regarding which tool to use. I only used nyc until now and as far as I know, it is still the most popular coverage tool for node packages. I think how the data is gathered is an implementation detail, and c8 still uses Istanbul for reports, so I don't think it makes much of a difference.