test: fix test-debugger-pid

The current format that prints the process PID before the actual message. Adapt the test so it also passes on Windows, that emits a different message from the rest of platforms. Move the test to parallel. PR-URL: #6584 Reviewed-By: Ben Noordhuis <[email protected]> Reviewed-By: Rich Trott <[email protected]>

Author: santigimeno

test/debugger/test-debugger-pid.js

@@ -0,0 +1,54 @@
+'use strict';
+constcommon=require('../common');
+constassert=require('assert');
+constspawn=require('child_process').spawn;
+
+varbuffer='';
+
+// connect to debug agent
+varinterfacer=spawn(process.execPath,['debug','-p','655555']);
+
+console.error(process.execPath,'debug','-p','655555');
+interfacer.stdout.setEncoding('utf-8');
+interfacer.stderr.setEncoding('utf-8');
+varonData=function(data){
+data=(buffer+data).split('\n');
+buffer=data.pop();
+data.forEach(function(line){
+interfacer.emit('line',line);
+});
+};
+interfacer.stdout.on('data',onData);
+interfacer.stderr.on('data',onData);
+
+varlineCount=0;
+interfacer.on('line',function(line){
+varexpected;
+constpid=interfacer.pid;
+if(common.isWindows){
+switch(++lineCount){
+case1:
+line=line.replace(/^(debug>*)+/,'');
+varmsg='There was an internal error in Node\'s debugger. '+
+'Please report this bug.';
+expected=`(node:${pid}) ${msg}`;
+break;
+
+case2:
+expected='The parameter is incorrect.';
+break;
+
+default:
+return;
+}
+}else{
+line=line.replace(/^(debug>*)+/,'');
+expected=`(node:${pid}) Target process: 655555 doesn\'t exist.`;
+}
+
+assert.strictEqual(expected,line);
+});
+
+interfacer.on('exit',function(code,signal){
+assert.ok(code==1,'Got unexpected code: '+code);
+});