test(exec): add tests for coverage by nfischer · Pull Request #744 · shelljs/shelljs
Conversation
No logic change.
This adds one test to cover some missing lines, and adds some istanbul ignore
directives.
I see 100% line coverage for src/exec.js when running:
$ nyc --reporter=text --reporter=lcov ava --serial test/exec.js`Fixes #742
No logic change. This adds one test to cover some missing lines, and adds some `istanbul ignore` directives. I see 100% line coverage for `src/exec.js` when running: ```sh $ nyc --reporter=text --reporter=lcov ava --serial test/exec.js` ``` Fixes #742
self-LGTM since this has no change to production logic.
This was referenced
Aug 29, 2019This was referenced
Mar 24, 2020This was referenced
Apr 24, 2020This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Labels
2 participants