From 0ec37414c2d80bd67121eb95085167a8b9333e30 Mon Sep 17 00:00:00 2001 From: Hornwitser Date: Wed, 4 Sep 2019 04:18:51 +0200 Subject: [PATCH] Deduplicate test script and check exit status Remove the explicit ignoring of the exit status of the test scripts. This makes the Travis CI build be considered a failure if one of the tests fail. Also rewrite the fast-test script to use the test script. --- package.json | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/package.json b/package.json index c53760a2..b217df62 100644 --- a/package.json +++ b/package.json @@ -5,10 +5,10 @@ "repository": "https://github.com/clusterio/factorioClusterio", "main": "master.js", "scripts": { - "test": "mocha --exit --check-leaks -R spec \"./{,!(node_modules)/**/}*.spec.js\" || true", - "fast-test": "FAST_TEST=y mocha --check-leaks --exit -R spec \"./{,!(node_modules)/**/}*.spec.js\" || true", - "cover": "nyc npm run-script test", - "fast-cover": "nyc npm run-script fast-test", + "test": "mocha --exit --check-leaks -R spec \"./{,!(node_modules)/**/}*.spec.js\"", + "fast-test": "FAST_TEST=y npm test", + "cover": "nyc npm test", + "fast-cover": "FAST_TEST=y nyc npm test", "ci-cover": "nyc --reporter=lcovonly npm run-script test", "start": "echo Run node client.js start [instance] to start slaves && node master.js", "install": "node lib/npmPostinstall.js",