test: split watch mode inspector tests to sequential

PR-URL: #44551
Reviewed-By: Kohei Ueno <kohei.ueno119@gmail.com>
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
Reviewed-By: Nitzan Uziely <linkgoron@gmail.com>

Author: MoLow

test/parallel/test-watch-mode.mjs

@@ -10,8 +10,6 @@ import { writeFileSync, readFileSync } from 'node:fs';
 import{inspect}from'node:util';
 import{once}from'node:events';
 import{setTimeout}from'node:timers/promises';
-import{NodeInstance}from'../common/inspector-helper.js';
-
 
 if(common.isIBMi)
 common.skip('IBMi does not support `fs.watch()`');
@@ -236,67 +234,4 @@ describe('watch mode', { concurrency: false, timeout: 60_0000 }, () => {
 `Completed running ${inspect(file)}`,`Restarting ${inspect(file)}`,`Completed running ${inspect(file)}`,'',
 ].join('\n'));
 });
-
-describe('inspect',{
-skip: Boolean(process.config.variables.coverage||!process.features.inspector),
-},()=>{
-constsilentLogger={log: ()=>{},error: ()=>{}};
-asyncfunctiongetDebuggedPid(instance,waitForLog=true){
-constsession=awaitinstance.connectInspectorSession();
-awaitsession.send({method: 'Runtime.enable'});
-if(waitForLog){
-awaitsession.waitForConsoleOutput('log','safe to debug now');
-}
-const{value: innerPid}=(awaitsession.send({
-'method': 'Runtime.evaluate','params': {'expression': 'process.pid'}
-})).result;
-session.disconnect();
-returninnerPid;
-}
-
-it('should start debugger on inner process',async()=>{
-constfile=fixtures.path('watch-mode/inspect.js');
-constinstance=newNodeInstance(['--inspect=0','--watch'],undefined,file,silentLogger);
-letstderr='';
-instance.on('stderr',(data)=>{stderr+=data;});
-
-constpids=[instance.pid];
-pids.push(awaitgetDebuggedPid(instance));
-instance.resetPort();
-writeFileSync(file,readFileSync(file));
-pids.push(awaitgetDebuggedPid(instance));
-
-awaitinstance.kill();
-
-// There should be 3 pids (one parent + 2 restarts).
-// Message about Debugger should only appear twice.
-assert.strictEqual(stderr.match(/Debuggerlisteningonws:\/\//g).length,2);
-assert.strictEqual(newSet(pids).size,3);
-});
-
-it('should prevent attaching debugger with SIGUSR1 to outer process',{skip: common.isWindows},async()=>{
-constfile=fixtures.path('watch-mode/inspect_with_signal.js');
-constinstance=newNodeInstance(['--inspect-port=0','--watch'],undefined,file,silentLogger);
-letstderr='';
-instance.on('stderr',(data)=>{stderr+=data;});
-
-constloggedPid=awaitnewPromise((resolve)=>{
-instance.on('stdout',(data)=>{
-constmatches=data.match(/pidis(\d+)/);
-if(matches)resolve(Number(matches[1]));
-});
-});
-
-
-process.kill(instance.pid,'SIGUSR1');
-process.kill(loggedPid,'SIGUSR1');
-constdebuggedPid=awaitgetDebuggedPid(instance,false);
-
-awaitinstance.kill();
-
-// Message about Debugger should only appear once in inner process.
-assert.strictEqual(stderr.match(/Debuggerlisteningonws:\/\//g).length,1);
-assert.strictEqual(loggedPid,debuggedPid);
-});
-});
 });

test/sequential/test-watch-mode-inspect.mjs

@@ -0,0 +1,73 @@
+import*ascommonfrom'../common/index.mjs';
+import*asfixturesfrom'../common/fixtures.mjs';
+importassertfrom'node:assert';
+import{describe,it}from'node:test';
+import{writeFileSync,readFileSync}from'node:fs';
+import{NodeInstance}from'../common/inspector-helper.js';
+
+
+if(common.isIBMi)
+common.skip('IBMi does not support `fs.watch()`');
+
+common.skipIfInspectorDisabled();
+
+describe('watch mode - inspect',()=>{
+constsilentLogger={log: ()=>{},error: ()=>{}};
+asyncfunctiongetDebuggedPid(instance,waitForLog=true){
+constsession=awaitinstance.connectInspectorSession();
+awaitsession.send({method: 'Runtime.enable'});
+if(waitForLog){
+awaitsession.waitForConsoleOutput('log','safe to debug now');
+}
+const{value: innerPid}=(awaitsession.send({
+'method': 'Runtime.evaluate','params': {'expression': 'process.pid'}
+})).result;
+session.disconnect();
+returninnerPid;
+}
+
+it('should start debugger on inner process',async()=>{
+constfile=fixtures.path('watch-mode/inspect.js');
+constinstance=newNodeInstance(['--inspect=0','--watch'],undefined,file,silentLogger);
+letstderr='';
+instance.on('stderr',(data)=>{stderr+=data;});
+
+constpids=[instance.pid];
+pids.push(awaitgetDebuggedPid(instance));
+instance.resetPort();
+writeFileSync(file,readFileSync(file));
+pids.push(awaitgetDebuggedPid(instance));
+
+awaitinstance.kill();
+
+// There should be 3 pids (one parent + 2 restarts).
+// Message about Debugger should only appear twice.
+assert.strictEqual(stderr.match(/Debuggerlisteningonws:\/\//g).length,2);
+assert.strictEqual(newSet(pids).size,3);
+});
+
+it('should prevent attaching debugger with SIGUSR1 to outer process',{skip: common.isWindows},async()=>{
+constfile=fixtures.path('watch-mode/inspect_with_signal.js');
+constinstance=newNodeInstance(['--inspect-port=0','--watch'],undefined,file,silentLogger);
+letstderr='';
+instance.on('stderr',(data)=>{stderr+=data;});
+
+constloggedPid=awaitnewPromise((resolve)=>{
+instance.on('stdout',(data)=>{
+constmatches=data.match(/pidis(\d+)/);
+if(matches)resolve(Number(matches[1]));
+});
+});
+
+
+process.kill(instance.pid,'SIGUSR1');
+process.kill(loggedPid,'SIGUSR1');
+constdebuggedPid=awaitgetDebuggedPid(instance,false);
+
+awaitinstance.kill();
+
+// Message about Debugger should only appear once in inner process.
+assert.strictEqual(stderr.match(/Debuggerlisteningonws:\/\//g).length,1);
+assert.strictEqual(loggedPid,debuggedPid);
+});
+});