test: improve code coverage for inspector connection errors

PR-URL: #42310 Refs: https://coverage.nodejs.org/coverage-457567f72ca73ef0/lib/inspector.js.html#L74 Reviewed-By: Darshan Sen <[email protected]> Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: James M Snell <[email protected]> Reviewed-By: Michael Dawson <[email protected]>

Author: cola119

test/parallel/test-inspector-connect-main-thread.js

@@ -8,7 +8,7 @@ const{Session } = require('inspector');
 constpath=require('path');
 const{ pathToFileURL }=require('url');
 const{ isMainThread, parentPort, Worker, workerData }=
-require('worker_threads');
+require('worker_threads');
 
 if(!workerData){
 common.skipIfWorker();
@@ -100,6 +100,17 @@ async function ensureListenerDoesNotInterrupt(session){
 }
 
 asyncfunctionmain(){
+assert.throws(
+()=>{
+constsession=newSession();
+session.connectToMainThread();
+},
+{
+code: 'ERR_INSPECTOR_NOT_WORKER',
+name: 'Error',
+message: 'Current thread is not a worker'
+}
+);
 constsharedBuffer=newSharedArrayBuffer(1);
 constarrayBuffer=newUint8Array(sharedBuffer);
 arrayBuffer[0]=1;
@@ -121,6 +132,16 @@ async function childMain(){
 await(awaitstartWorker(true)).onMessagesSent;
 constsession=newSession();
 session.connectToMainThread();
+assert.throws(
+()=>{
+session.connectToMainThread();
+},
+{
+code: 'ERR_INSPECTOR_ALREADY_CONNECTED',
+name: 'Error',
+message: 'The inspector session is already connected'
+}
+);
 awaitpost(session,'Debugger.enable');
 awaitpost(session,'Runtime.enable');
 awaitpost(session,'Debugger.setBreakpointByUrl',{
@@ -137,9 +158,9 @@ async function childMain(){
 awaitnewPromise((resolve)=>setTimeout(resolve,50));
 
 const{result: { value }}=
-awaitpost(session,
-'Debugger.evaluateOnCallFrame',
-{ callFrameId,expression: 'a * 100'});
+awaitpost(session,
+'Debugger.evaluateOnCallFrame',
+{ callFrameId,expression: 'a * 100'});
 assert.strictEqual(value,100);
 awaitpost(session,'Debugger.resume');
 awaitensureListenerDoesNotInterrupt(session);