Re-enable test coverage reporting

Fixed the cov command just like I'd done for the test command, so that it will properly exit with an non-zero exit code on test failure. Switched back to using the cov script for doing the tests.
breser · Aug 12, 2017 · e6f4902 · e6f4902
1 parent 029102f
commit e6f4902
Show file tree

Hide file tree

Showing 3 changed files with 11 additions and 3 deletions.
diff --git a/.travis.yml b/.travis.yml
@@ -10,7 +10,7 @@ before_script:
 - git config --global user.email "ryan@ryanbreen.com"
 - git config --global user.name "Ryan Breen"
 script:
-- npm run test
+- npm run cov
 deploy:
   provider: npm
   email: ryan@ryanbreen.com

diff --git a/package.json b/package.json
@@ -38,7 +38,7 @@
   "bin": "./index.js",
   "scripts": {
     "test": "./test/test.sh",
-    "cov": "`npm bin`/istanbul cover --root . -x node_modules -x test --dir ./reports `npm bin`/_mocha -- --reporter spec -t 10000 test | bunyan",
+    "cov": "./test/test.sh cov",
     "coveralls": "npm run cov && node_modules/coveralls/bin/coveralls.js < reports/lcov.info"
   },
   "licenses": [

diff --git a/test/test.sh b/test/test.sh
@@ -2,4 +2,12 @@
 
 set -o pipefail
 
-./node_modules/.bin/_mocha --reporter spec -t 10000 test | bunyan
+bin=$(npm bin)
+mocha_args='--reporter spec -t 10000 test'
+
+if [ "$1" == "cov" ]; then
+  "$bin/istanbul" cover --root . -x node_modules -x test --dir ./reports \
+    "$bin/_mocha" -- $mocha_args | bunyan
+else
+  "$bin/_mocha" $mocha_args | bunyan
+fi