Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Measure Unit Test Coverage #3385

Closed
outofambit opened this issue Dec 11, 2018 · 5 comments · Fixed by #3834
Closed

Measure Unit Test Coverage #3385

outofambit opened this issue Dec 11, 2018 · 5 comments · Fixed by #3834
Assignees
Milestone

Comments

@outofambit
Copy link
Contributor

Instrument the unit test suite so we can see which parts of the codebase are well tested and which need more coverage.

Some relevant libraries and services that might work for this:

@vipulrawat
Copy link
Member

vipulrawat commented Dec 12, 2018

One thing, but might not be preferable as p5.js is already built up on mocha.
If p5.js is targeting a whole new set of paradigm with all es6 features, then one can think on migrating to jest.
Reason being:

@rakeshcheekatimala
Copy link

I want to join as a contributor, I have a basic understanding of Jest and writing Unit Test Cases. Please let me know your thoughts.

@outofambit
Copy link
Contributor Author

I started working on this and am hitting a number of unexpected bumps to completing it. (Currently, I am fighting with instrumenting the code and uglifyjs's syntax parsing limitations.) Will update as things progress! 🚀

@lmccart lmccart modified the milestones: 0.8.0, 0.9.0 Apr 8, 2019
@ihsavru
Copy link
Member

ihsavru commented Apr 25, 2019

Hi @outofambit, I was hoping this can be completed before writing more unit tests. I would like to help with this.

@outofambit outofambit self-assigned this Jun 10, 2019
@outofambit
Copy link
Contributor Author

been researching this and why the standard https://github.com/tandrewnichols/grunt-simple-nyc doesn't seem to work for us. i will have to do something similar to the approach described in this tutorial: https://www.engineyard.com/blog/measuring-clientside-javascript-test-coverage-with-istanbul

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants