test: improve test-debugger-util-regression

Avoid the `exit` command to be sent more than once. It prevents from undesired errors emitted on `proc.stdin`. Remove the watchdog timer so the test does not fail in case it takes longer to complete. PR-URL: #9490 Reviewed-By: Rich Trott <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: James Snell <[email protected]>

Author: santigimeno

test/parallel/test-debugger-util-regression.js

@@ -19,16 +19,11 @@ const proc = spawn(process.execPath, args,{stdio: 'pipe'});
 proc.stdout.setEncoding('utf8');
 proc.stderr.setEncoding('utf8');
 
-functionfail(){
-common.fail('the program should not hang');
-}
-
-consttimer=setTimeout(fail,common.platformTimeout(4000));
-
 letstdout='';
 letstderr='';
 
 letnextCount=0;
+letexit=false;
 
 proc.stdout.on('data',(data)=>{
 stdout+=data;
@@ -38,8 +33,8 @@ proc.stdout.on('data', (data) =>{
 stdout.includes('> 4')&&nextCount<4){
 nextCount++;
 proc.stdin.write('n\n');
-}elseif(stdout.includes('{a: \'b\' }')){
-clearTimeout(timer);
+}elseif(!exit&&(stdout.includes('< {a: \'b\' }'))){
+exit=true;
 proc.stdin.write('.exit\n');
 }elseif(stdout.includes('program terminated')){
 // Catch edge case present in v4.x
@@ -50,15 +45,6 @@ proc.stdout.on('data', (data) =>{
 
 proc.stderr.on('data',(data)=>stderr+=data);
 
-// FIXME
-// This test has been periodically failing on certain systems due to
-// uncaught errors on proc.stdin. This will stop the process from
-// exploding but is still not an elegant solution. Likely a deeper bug
-// causing this problem.
-proc.stdin.on('error',(err)=>{
-console.error(err);
-});
-
 process.on('exit',(code)=>{
 assert.equal(code,0,'the program should exit cleanly');
 assert.equal(stdout.includes('{a: \'b\'}'),true,