From 044a0be46f09844b3e9d6aa68a0e8125224ecfa6 Mon Sep 17 00:00:00 2001
From: Rich Trott <rtrott@gmail.com>
Date: Wed, 22 Jun 2016 16:21:21 -0700
Subject: [PATCH] test: add test for exec() known issue
PR-URL: https://github.com/nodejs/node/pull/7375
Refs: https://github.com/nodejs/node/issues/7342
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Santiago Gimeno <santiago.gimeno@gmail.com>
---
...test-child-process-exec-stdout-data-string.js | 16 ++++++++++++++++
1 file changed, 16 insertions(+)
create mode 100644 test/known_issues/test-child-process-exec-stdout-data-string.js
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
new file mode 100644
index 00000000000000..b267ff5e98347a
--- /dev/null
+++ 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);