Skip to content
This repository has been archived by the owner on Mar 25, 2021. It is now read-only.

Commit

Permalink
Tweak tests/build so they more reliably pass on all machines (#1180)
Browse files Browse the repository at this point in the history
* Remove test relying on `mktemp -d`
* Clean tests' node_modules so a clean install happens on each test run
  • Loading branch information
jkillian committed Apr 29, 2016
1 parent 946ab3c commit d4e227a
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 9 deletions.
2 changes: 1 addition & 1 deletion Gruntfile.js
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@ module.exports = function (grunt) {

clean: {
core: ["lib/**/*.js", "lib/**/*.d.ts"],
test: ["build/"]
test: ["build/", "test/config/node_modules/"]
},

mochaTest: {
Expand Down
8 changes: 0 additions & 8 deletions test/check-bin.sh
Original file line number Diff line number Diff line change
Expand Up @@ -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
Expand Down

0 comments on commit d4e227a

Please sign in to comment.