Skip to content

Commit

Permalink
add Node.js 8 to Travis CI (#2086)
Browse files Browse the repository at this point in the history
- explicitly terminate `test/jetstream.js` upon completion
- log verbose output from `test/benchmark.js` & `test/jetstream.js`
- remove obsolete workaround for Travis CI
  • Loading branch information
alexlamsl committed Jun 12, 2017
1 parent fed0096 commit 3dc9e14
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 5 deletions.
1 change: 1 addition & 0 deletions .travis.yml
Expand Up @@ -4,6 +4,7 @@ node_js:
- "0.12"
- "4"
- "6"
- "8"
env:
- UGLIFYJS_TEST_ALL=1
matrix:
Expand Down
1 change: 1 addition & 0 deletions test/jetstream.js
Expand Up @@ -64,6 +64,7 @@ if (typeof phantom == "undefined") {
server.close();
if (code) throw new Error("JetStream failed!");
console.log("JetStream completed successfully.");
process.exit(0);
});
});
}
Expand Down
6 changes: 1 addition & 5 deletions test/mocha/release.js
Expand Up @@ -4,13 +4,9 @@ var spawn = require("child_process").spawn;
if (!process.env.UGLIFYJS_TEST_ALL) return;

function run(command, args, done) {
var id = setInterval(function() {
process.stdout.write("\0");
}, 5 * 60 * 1000);
spawn(command, args, {
stdio: "ignore"
stdio: [ "ignore", 1, 2 ]
}).on("exit", function(code) {
clearInterval(id);
assert.strictEqual(code, 0);
done();
});
Expand Down

0 comments on commit 3dc9e14

Please sign in to comment.