From d4e227a1337586786ae47ad590ad52b89957e7c6 Mon Sep 17 00:00:00 2001 From: Jason Killian Date: Fri, 29 Apr 2016 00:28:24 -0400 Subject: [PATCH] Tweak tests/build so they more reliably pass on all machines (#1180) * Remove test relying on `mktemp -d` * Clean tests' node_modules so a clean install happens on each test run --- Gruntfile.js | 2 +- test/check-bin.sh | 8 -------- 2 files changed, 1 insertion(+), 9 deletions(-) diff --git a/Gruntfile.js b/Gruntfile.js index 6aae526aece..308023c7188 100644 --- a/Gruntfile.js +++ b/Gruntfile.js @@ -14,7 +14,7 @@ module.exports = function (grunt) { clean: { core: ["lib/**/*.js", "lib/**/*.d.ts"], - test: ["build/"] + test: ["build/", "test/config/node_modules/"] }, mochaTest: { diff --git a/test/check-bin.sh b/test/check-bin.sh index eb831206ba8..254e0658ad0 100755 --- a/test/check-bin.sh +++ b/test/check-bin.sh @@ -74,14 +74,6 @@ expectOut $? 0 "-c relative path without ./ did not work" ./bin/tslint -c test/config/tslint-extends-package-no-mod.json src/tslint.ts expectOut $? 0 "tslint (with config file extending relative package) did not work" -# check that a tslint file (outside of the resolution path of the tslint package) can resolve a package that is -# installed as a dependency of tslint. See palantir/tslint#1172 for details. -tmpDir=$(mktemp -d) -echo "{ \"extends\": \"tslint-test-config-non-relative\" }" > $tmpDir/tslint.json -./bin/tslint -c $tmpDir/tslint.json src/tslint.ts -expectOut $? 0 "tslint (with config file extending package in tslint scope) did not work" -rm -r $tmpDir - # make sure tslint --init generates a file cd ./bin if [ -f tslint.json ]; then