test(index): cover spawn errors #732
Conversation
There was a problem hiding this comment.
Pull request overview
This PR improves test coverage for the Poppler constructor by refining Istanbul ignore comments to more precisely identify OS-specific code paths while enabling coverage of the error handling logic.
- Removed ignore comments from testable code paths (constructor binPath check and error throw)
- Repositioned ignore comments to precisely mark OS-specific operations (spawnSync and Windows-specific fallback)
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
You can also share your feedback on Copilot code review for a chance to win a $100 gift card. Take the survey.
Fdawgs
force-pushed
the
test/coverage
branch
from
2f33982 to
b7a1213
Compare
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
You can also share your feedback on Copilot code review for a chance to win a $100 gift card. Take the survey.
Checklist
npm test