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

🛠️ Repo: Increase unit test coverage #3601

Open
2 tasks
boneskull opened this issue Dec 8, 2018 · 0 comments
Open
2 tasks

🛠️ Repo: Increase unit test coverage #3601

boneskull opened this issue Dec 8, 2018 · 0 comments
Labels
area: repository tooling concerning ease of contribution status: accepting prs Mocha can use your help with this one!

Comments

@boneskull
Copy link
Contributor

boneskull commented Dec 8, 2018

counting our integration tests, our coverage is hovering around 90% at time of writing.

...but if we're just looking at unit test coverage, it's only about 50%. 😬

Two parts to this:

  • Set up Mocha to compute unit test coverage in isolation
    Create a script (package-scripts.js) and/or configure nyc (.nycrc) to easily compute unit test coverage

    Don't include stuff that we can't unit test reasonably within the count. If it isn't require'd by anything, that's a hint. If the coverage doesn't stand at about 50%, then too much is getting counted.

    This may necessitate two separate setups for browser and Node.js. Just figure out Node.js for this issue; see 🛠️ Repo: Increase browser coverage #2532 if you want to figure out how to compute browser coverage

  • Write moar tests (in test/unit/)

@boneskull boneskull added status: accepting prs Mocha can use your help with this one! qa labels Dec 8, 2018
@JoshuaKGoldberg JoshuaKGoldberg changed the title increase unit test coverage 🛠️ Repo: Increase unit test coverage Dec 27, 2023
@JoshuaKGoldberg JoshuaKGoldberg added area: repository tooling concerning ease of contribution and removed area: qa labels Jan 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: repository tooling concerning ease of contribution status: accepting prs Mocha can use your help with this one!
Projects
None yet
Development

No branches or pull requests

2 participants