test: move some test from sequential to parallel

The only test with modifications is `test-stdin-child-proc` that was
passing when it should not because the exit code of the child process
was not being checked.

PR-URL: https://github.com/nodejs/node/pull/6087
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Johan Bergström <bugs@bergstroem.nu>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Sakthipriyan Vairamani <thechargingvolcano@gmail.com>
Reviewed-By: Claudio Rodriguez <cjrodr@yahoo.com>

20 files changed, 13 insertions, 8 deletions

diff --git a/test/sequential/test-buffer-bad-overload.js b/test/parallel/test-buffer-bad-overload.js
index 3d99dd532e..3d99dd532e 100644
--- a/test/sequential/test-buffer-bad-overload.js
+++ b/test/parallel/test-buffer-bad-overload.js
diff --git a/test/sequential/test-debug-args.js b/test/parallel/test-debug-args.js
index e5f9f37161..e5f9f37161 100644
--- a/test/sequential/test-debug-args.js
+++ b/test/parallel/test-debug-args.js
diff --git a/test/sequential/test-force-repl-with-eval.js b/test/parallel/test-force-repl-with-eval.js
index a285db8e57..a285db8e57 100644
--- a/test/sequential/test-force-repl-with-eval.js
+++ b/test/parallel/test-force-repl-with-eval.js
diff --git a/test/sequential/test-force-repl.js b/test/parallel/test-force-repl.js
index 5907dc2019..5907dc2019 100644
--- a/test/sequential/test-force-repl.js
+++ b/test/parallel/test-force-repl.js
diff --git a/test/sequential/test-net-localport.js b/test/parallel/test-net-localport.js
index 09b6627c79..09b6627c79 100644
--- a/test/sequential/test-net-localport.js
+++ b/test/parallel/test-net-localport.js
diff --git a/test/sequential/test-regress-GH-1531.js b/test/parallel/test-regress-GH-1531.js
index 097611e1bf..097611e1bf 100644
--- a/test/sequential/test-regress-GH-1531.js
+++ b/test/parallel/test-regress-GH-1531.js
diff --git a/test/sequential/test-regress-GH-1899.js b/test/parallel/test-regress-GH-1899.js
index 98396aae13..98396aae13 100644
--- a/test/sequential/test-regress-GH-1899.js
+++ b/test/parallel/test-regress-GH-1899.js
diff --git a/test/sequential/test-regress-GH-4948.js b/test/parallel/test-regress-GH-4948.js
index c6953eb78f..c6953eb78f 100644
--- a/test/sequential/test-regress-GH-4948.js
+++ b/test/parallel/test-regress-GH-4948.js
diff --git a/test/sequential/test-regress-GH-746.js b/test/parallel/test-regress-GH-746.js
index 4b79f3c4e3..4b79f3c4e3 100644
--- a/test/sequential/test-regress-GH-746.js
+++ b/test/parallel/test-regress-GH-746.js
diff --git a/test/sequential/test-socket-write-after-fin-error.js b/test/parallel/test-socket-write-after-fin-error.js
index 226440ec50..226440ec50 100644
--- a/test/sequential/test-socket-write-after-fin-error.js
+++ b/test/parallel/test-socket-write-after-fin-error.js
diff --git a/test/sequential/test-socket-write-after-fin.js b/test/parallel/test-socket-write-after-fin.js
index ea8ac27b59..ea8ac27b59 100644
--- a/test/sequential/test-socket-write-after-fin.js
+++ b/test/parallel/test-socket-write-after-fin.js
diff --git a/test/parallel/test-stdin-child-proc.js b/test/parallel/test-stdin-child-proc.js
new file mode 100644
index 0000000000..35ae0c99d3
--- /dev/null
+++ b/test/parallel/test-stdin-child-proc.js
@@ -0,0 +1,13 @@
+'use strict';
+// This tests that pausing and resuming stdin does not hang and timeout
+// when done in a child process.  See test/parallel/test-stdin-pause-resume.js
+const common = require('../common');
+const assert = require('assert');
+const child_process = require('child_process');
+const path = require('path');
+const cp = child_process.spawn(process.execPath,
+                    [path.resolve(__dirname, 'test-stdin-pause-resume.js')]);
+
+cp.on('exit', common.mustCall((code) => {
+  assert.equal(code, 0);
+}));
diff --git a/test/sequential/test-stdin-pipe-resume.js b/test/parallel/test-stdin-pipe-resume.js
index cd409a1d23..cd409a1d23 100644
--- a/test/sequential/test-stdin-pipe-resume.js
+++ b/test/parallel/test-stdin-pipe-resume.js
diff --git a/test/sequential/test-stdin-script-child.js b/test/parallel/test-stdin-script-child.js
index ad65734364..ad65734364 100644
--- a/test/sequential/test-stdin-script-child.js
+++ b/test/parallel/test-stdin-script-child.js
diff --git a/test/sequential/test-stdout-cannot-be-closed-child-process-pipe.js b/test/parallel/test-stdout-cannot-be-closed-child-process-pipe.js
index a138108fae..a138108fae 100644
--- a/test/sequential/test-stdout-cannot-be-closed-child-process-pipe.js
+++ b/test/parallel/test-stdout-cannot-be-closed-child-process-pipe.js
diff --git a/test/sequential/test-stdout-to-file.js b/test/parallel/test-stdout-to-file.js
index bd1d97b3d8..bd1d97b3d8 100644
--- a/test/sequential/test-stdout-to-file.js
+++ b/test/parallel/test-stdout-to-file.js
diff --git a/test/sequential/test-sync-fileread.js b/test/parallel/test-sync-fileread.js
index 6208e4c619..6208e4c619 100644
--- a/test/sequential/test-sync-fileread.js
+++ b/test/parallel/test-sync-fileread.js
diff --git a/test/sequential/test-vm-syntax-error-stderr.js b/test/parallel/test-vm-syntax-error-stderr.js
index 7c3c5ff135..7c3c5ff135 100644
--- a/test/sequential/test-vm-syntax-error-stderr.js
+++ b/test/parallel/test-vm-syntax-error-stderr.js
diff --git a/test/sequential/test-zerolengthbufferbug.js b/test/parallel/test-zerolengthbufferbug.js
index 88e66a8d85..88e66a8d85 100644
--- a/test/sequential/test-zerolengthbufferbug.js
+++ b/test/parallel/test-zerolengthbufferbug.js
diff --git a/test/sequential/test-stdin-child-proc.js b/test/sequential/test-stdin-child-proc.js
deleted file mode 100644
index 0189e94d5d..0000000000
--- a/test/sequential/test-stdin-child-proc.js
+++ /dev/null
@@ -1,8 +0,0 @@
-'use strict';
-// This tests that pausing and resuming stdin does not hang and timeout
-// when done in a child process.  See test/simple/test-stdin-pause-resume.js
-require('../common');
-var child_process = require('child_process');
-var path = require('path');
-child_process.spawn(process.execPath,
-                    [path.resolve(__dirname, 'test-stdin-pause-resume.js')]);