chore: create exclusions for nyc

[lennonnikolas]

PR Checklist

• Addresses an existing open issue: fixes 🛠 Repo: Code coverage includes package-lock.json #5558
• That issue was marked as status: accepting prs
• Steps in CONTRIBUTING.md were taken

Overview

Will now exclude package-lock.json, node_modules, and test directory from nyc coverage

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 87.98%. Comparing base (1544c39) to head (8462a69).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #5609      +/-   ##
==========================================
- Coverage   93.53%   87.98%   -5.55%     
==========================================
  Files          57       67      +10     
  Lines        4472     4754     +282     
  Branches      922      964      +42     
==========================================
  Hits         4183     4183              
- Misses        289      571     +282     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[mark-wiemer]

LGTM, @lennonnikolas let me know if there is anything preventing us from merging this one :)

[lennonnikolas]

LGTM, @lennonnikolas let me know if there is anything preventing us from merging this one :)

Nothing out of the ordinary for me!

[jedwards1211]

@mark-wiemer @lennonnikolas this PR added an nyc stanza in package.json instead of modifying or replacing the existing .nycrc file, I think this was a mistake? Shouldn't there only be one or the other? I'm pretty sure package.json stanza overrides the separate config files, though nyc's docs don't say for sure. There are many more exclusions in the existing .nycrc than what was added in this PR.

[lennonnikolas]

@mark-wiemer @lennonnikolas this PR added an nyc stanza in package.json instead of modifying or replacing the existing .nycrc file, I think this was a mistake? Shouldn't there only be one or the other? I'm pretty sure package.json stanza overrides the separate config files, though nyc's docs don't say for sure. There are many more exclusions in the existing .nycrc than what was added in this PR.

Yeah the world of never-ending configurations. To be honest I overlooked this file. I'm newer to open source development. We can definitely migrate to the already existing RC file.