search

josdejong / workerpool

Offload tasks to a pool of workers on node.js and in the browser
Apache License 2.0
2.06k stars 147 forks source link

Use c8 for test coverage #385

Closed brunoSnoww closed 1 year ago

brunoSnoww commented 1 year ago

This PR addresses https://github.com/josdejong/workerpool/issues/360. It replaces the istanbul test coverage package to c8. It generates a .html report under ./coverage for all test files

josdejong commented 1 year ago

Thanks Bruno!

When I run npm run coverage (on Windows), I get an error: 

Error: Cannot find module 'html;'

All works when I replace c8 report --reporter=html; with c8 report --reporter=html &&, can you adjust that in the PR?

brunoSnoww commented 1 year ago

thanks for pointing that out ! 😬

josdejong commented 1 year ago

Thanks, looks good to go Bruno 👌