diff options
author | Alex Lam S.L <alexlamsl@gmail.com> | 2019-05-12 09:44:02 +0800 |
---|---|---|
committer | GitHub <noreply@github.com> | 2019-05-12 09:44:02 +0800 |
commit | 45fbdbc2dcede8115687ae02619854ed974c7d1f (patch) | |
tree | f3306f43cdfe86a61c33c98e470c1b574568f6b0 | |
parent | 54cb67805532667af50947cc3846d2cacc2a0a1d (diff) | |
download | tracifyjs-45fbdbc2dcede8115687ae02619854ed974c7d1f.tar.gz tracifyjs-45fbdbc2dcede8115687ae02619854ed974c7d1f.zip |
improve tests (#3408)
-rw-r--r-- | test/jetstream.js | 13 | ||||
-rw-r--r-- | test/release/run.js | 4 |
2 files changed, 13 insertions, 4 deletions
diff --git a/test/jetstream.js b/test/jetstream.js index 2d85dfdd..e7c846e6 100644 --- a/test/jetstream.js +++ b/test/jetstream.js @@ -62,8 +62,17 @@ if (typeof phantom == "undefined") { if (debug) { console.log("http://localhost:" + port + "/"); } else { - child_process.exec("npm install phantomjs-prebuilt@2.1.14 --no-save", function(error) { - if (error) throw error; + child_process.spawn(process.platform == "win32" ? "npm.cmd" : "npm", [ + "install", + "phantomjs-prebuilt@2.1.14", + "--no-audit", + "--no-optional", + "--no-save", + "--no-update-notifier", + ], { + stdio: [ "ignore", 1, 2 ] + }).on("exit", function(code) { + if (code) throw new Error("npm install failed!"); var program = require("phantomjs-prebuilt").exec(process.argv[1], port); program.stdout.pipe(process.stdout); program.stderr.pipe(process.stderr); diff --git a/test/release/run.js b/test/release/run.js index 7b4755dc..5dc85193 100644 --- a/test/release/run.js +++ b/test/release/run.js @@ -6,10 +6,10 @@ module.exports = function(tasks) { var args = tasks.shift(); console.log(); console.log("\u001B[36m$> " + args.join(" ") + "\u001B[39m"); - var result = child_process.spawn(process.argv[0], args, { + child_process.spawn(process.argv[0], args, { stdio: [ "ignore", 1, 2 ] }).on("exit", function(code) { - if (code != 0) process.exit(code); + if (code) process.exit(code); next(); }); })(); |