test: fix flaky repl-timeout-throw

Don't disconnect the child until all exceptions are thrown. Fixes: #18659 PR-URL: #18692 Fixes: #18659 Reviewed-By: Ben Noordhuis <[email protected]> Reviewed-By: Ruben Bridgewater <[email protected]> Reviewed-By: Colin Ihrig <[email protected]>
nodejs · Feb 21, 2018 · 28207e3 · 28207e3
1 parent 38451b9
commit 28207e3
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/test/sequential/test-repl-timeout-throw.js b/test/sequential/test-repl-timeout-throw.js
@@ -1,5 +1,5 @@
 'use strict';
-const common = require('../common');
+require('../common');
 const assert = require('assert');
 
 const spawn = require('child_process').spawn;
@@ -13,6 +13,8 @@ child.stdout.setEncoding('utf8');
 child.stdout.on('data', function(c) {
   process.stdout.write(c);
   stdout += c;
+  if (stdout.includes('> THROW 2'))
+    child.stdin.end();
 });
 
 child.stdin.write = function(original) {
@@ -46,8 +48,6 @@ child.stdout.once('data', function() {
                       '    });\n' +
                       '  });\n' +
                       '});"";\n');
-
-    setTimeout(child.stdin.end.bind(child.stdin), common.platformTimeout(200));
   }
 });