From 070b63bb3b1da8c699401066f780bb4efdab7ed1 Mon Sep 17 00:00:00 2001 From: Rich Trott Date: Fri, 1 Sep 2017 17:30:10 -0700 Subject: [PATCH] test: refactor test-debug-prompt * Use cleaner `process.stdin.write('.exit')` to exit the process rather than `proc.kill()`. * Move test to sequential. It uses the default port 9229. It will fail if another inspector test (or test using port 0) is already using that port. So it needs to be run sequentially rather than in parallel with other tests. (We haven't seen many failures with it yet because there aren't a lot of other inspector tests in parallel at this time.) --- test/{parallel => sequential}/test-debug-prompt.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) rename test/{parallel => sequential}/test-debug-prompt.js (90%) diff --git a/test/parallel/test-debug-prompt.js b/test/sequential/test-debug-prompt.js similarity index 90% rename from test/parallel/test-debug-prompt.js rename to test/sequential/test-debug-prompt.js index 81e579ed1e2047..902bce34fc46f4 100644 --- a/test/parallel/test-debug-prompt.js +++ b/test/sequential/test-debug-prompt.js @@ -11,5 +11,5 @@ let output = ''; proc.stdout.on('data', (data) => { output += data; if (output.includes('debug> ')) - proc.kill(); + proc.stdin.write('.exit\n'); });