test: add test for exec() known issue

Trott · Fishrock123 · commit 16aff79deea7 · 2016-07-05T22:11:45.000+02:00
PR-URL: #7375 Refs: #7342 Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Santiago Gimeno <santiago.gimeno@gmail.com>
diff --git a/test/known_issues/test-child-process-exec-stdout-data-string.js b/test/known_issues/test-child-process-exec-stdout-data-string.js
@@ -0,0 +1,16 @@
+'use strict';
+// Refs: https://github.com/nodejs/node/issues/7342
+const common = require('../common');
+const assert = require('assert');
+const exec = require('child_process').exec;
+
+const expectedCalls = 2;
+
+const cb = common.mustCall((data) =>{
+ assert.strictEqual(typeof data, 'string');
+}, expectedCalls);
+
+const command = common.isWindows ? 'dir' : 'ls';
+exec(command).stdout.on('data', cb);
+
+exec('fhqwhgads').stderr.on('data', cb);

-Original file line number
+Diff line change
@@ @@ -0,0 +1,16 @@ @@
 +'use strict';
 +// Refs: https://github.com/nodejs/node/issues/7342
 +constcommon=require('../common');
 +constassert=require('assert');
 +constexec=require('child_process').exec;
++
 +constexpectedCalls=2;
++
 +constcb=common.mustCall((data)=>{
 +assert.strictEqual(typeofdata,'string');
 +},expectedCalls);
++
 +constcommand=common.isWindows ? 'dir' : 'ls';
 +exec(command).stdout.on('data',cb);
++
 +exec('fhqwhgads').stderr.on('data',cb);