Merge pull request #29 from pelias/safely-run-tests

chore(test): Use dedicated test script to catch errors
pelias · Sep 13, 2018 · db72c98 · db72c98
2 parents 750f255 + fa24b40
commit db72c98
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 1 deletion.
diff --git a/bin/test b/bin/test
@@ -0,0 +1,7 @@
+#!/bin/bash
+
+# run tests with pipefail to avoid false passes
+# see https://github.com/pelias/pelias/issues/744
+set -euo pipefail
+
+node test/test | npx tap-dot
diff --git a/package.json b/package.json
@@ -4,7 +4,7 @@
   "description": "Module that provides a convenience wrapper around HTTP GET microservices",
   "main": "index.js",
   "scripts": {
-    "test": "node test/test | tap-dot",
+    "test": "./bin/test",
     "lint": "jshint .",
     "travis": "npm test",
     "validate": "npm ls"