test: improve tests in pummel/test-exec

* add asset.strictEqual to stdout and stderr. Fixes no-unused-vars. * replace double quotes with single PR-URL: #10757 Reviewed-By: Rich Trott <[email protected]> Reviewed-By: James M Snell <[email protected]> Reviewed-By: Gibson Fahnestock <[email protected]> Reviewed-By: Michael Dawson <[email protected]>
nodejs · Jan 27, 2017 · fca0da7 · fca0da7
1 parent d64e237
commit fca0da7
Showing 1 changed file with 6 additions and 0 deletions.
diff --git a/test/pummel/test-exec.js b/test/pummel/test-exec.js
@@ -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, '');
 });
 
 
@@ -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);
@@ -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, '');
      });