test: deflake test-esm-loader-hooks-inspect-brk

lpinca · aduh95 · commit 7819bfec69b5 · 2025-01-02T15:13:18.000+01:00
Refs: #54827 Refs: #51560 PR-URL: #56050 Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
diff --git a/test/parallel/test-esm-loader-hooks-inspect-brk.js b/test/parallel/test-esm-loader-hooks-inspect-brk.js
@@ -10,23 +10,20 @@ const assert = require('assert');
 const fixtures = require('../common/fixtures');
 const{NodeInstance } = require('../common/inspector-helper.js');
 
-async function runIfWaitingForDebugger(session){
- const commands = [
-{'method': 'Runtime.enable' },
-{'method': 'Debugger.enable' },
-{'method': 'Runtime.runIfWaitingForDebugger' },
- ];
-
- await session.send(commands);
- await session.waitForNotification('Debugger.paused');
-}
-
 async function runTest(){
 const main = fixtures.path('es-module-loaders', 'register-loader.mjs');
 const child = new NodeInstance(['--inspect-brk=0'], '', main);
 
 const session = await child.connectInspectorSession();
- await runIfWaitingForDebugger(session);
+ await session.send({method: 'NodeRuntime.enable' });
+ await session.waitForNotification('NodeRuntime.waitingForDebugger');
+ await session.send([
+{'method': 'Runtime.enable' },
+{'method': 'Debugger.enable' },
+{'method': 'Runtime.runIfWaitingForDebugger' },
+ ]);
+ await session.send({method: 'NodeRuntime.disable' });
+ await session.waitForNotification('Debugger.paused');
 await session.runToCompletion();
 assert.strictEqual((await child.expectShutdown()).exitCode, 0);
 }

-Original file line number
+Diff line change
 constfixtures=require('../common/fixtures');
 const{ NodeInstance }=require('../common/inspector-helper.js');
 -asyncfunctionrunIfWaitingForDebugger(session){
 -constcommands=[
 -{'method': 'Runtime.enable'},
 -{'method': 'Debugger.enable'},
 -{'method': 'Runtime.runIfWaitingForDebugger'},
 -];
+-
 -awaitsession.send(commands);
 -awaitsession.waitForNotification('Debugger.paused');
 -}
+-
 asyncfunctionrunTest(){
 constmain=fixtures.path('es-module-loaders','register-loader.mjs');
 constchild=newNodeInstance(['--inspect-brk=0'],'',main);
 constsession=awaitchild.connectInspectorSession();
 -awaitrunIfWaitingForDebugger(session);
 +awaitsession.send({method: 'NodeRuntime.enable'});
 +awaitsession.waitForNotification('NodeRuntime.waitingForDebugger');
 +awaitsession.send([
 +{'method': 'Runtime.enable'},
 +{'method': 'Debugger.enable'},
 +{'method': 'Runtime.runIfWaitingForDebugger'},
 +]);
 +awaitsession.send({method: 'NodeRuntime.disable'});
 +awaitsession.waitForNotification('Debugger.paused');
 awaitsession.runToCompletion();
 assert.strictEqual((awaitchild.expectShutdown()).exitCode,0);
+}