Skip to content

Commit

Permalink
test: improve tests in pummel/test-exec
Browse files Browse the repository at this point in the history
* add asset.strictEqual to stdout and stderr. Fixes no-unused-vars.
* replace double quotes with single

PR-URL: #10757
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Gibson Fahnestock <gibfahn@gmail.com>
Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com>
  • Loading branch information
chasestarr authored and MylesBorins committed Mar 8, 2017
1 parent 9aeb34a commit f68fb7c
Showing 1 changed file with 6 additions and 0 deletions.
6 changes: 6 additions & 0 deletions test/pummel/test-exec.js
Original file line number Diff line number Diff line change
Expand Up @@ -63,6 +63,8 @@ exec(SLEEP3_COMMAND, { timeout: 50 }, function(err, stdout, stderr) {
assert.ok(err);
assert.ok(err.killed);
assert.strictEqual(err.signal, 'SIGTERM');
assert.strictEqual(stdout, '');
assert.strictEqual(stderr, '');
});


Expand All @@ -85,6 +87,8 @@ function killMeTwiceCallback(err, stdout, stderr) {
assert.ok(err);
assert.ok(err.killed);
assert.strictEqual(err.signal, 'SIGTERM');
assert.strictEqual(stdout, '');
assert.strictEqual(stderr, '');

// the timeout should still be in effect
console.log('\'sleep 3\' was already killed. Took %d ms', diff);
Expand All @@ -96,6 +100,8 @@ exec('python -c "print 200000*\'C\'"', {maxBuffer: 1000},
function(err, stdout, stderr) {
assert.ok(err);
assert.ok(/maxBuffer/.test(err.message));
assert.strictEqual(stdout, '');
assert.strictEqual(stderr, '');
});


Expand Down

0 comments on commit f68fb7c

Please sign in to comment.