test: use common.skipIfInspectorDisabled() to skip tests

some test files manually check for `!process.features.inspector`
to see if the tests should be skipped, the changes here update
those checks to instead use the more appropriate
`common.skipIfInspectorDisabled()`

PR-URL: #58675
Reviewed-By: Darshan Sen <raisinten@gmail.com>
Reviewed-By: Chengzhong Wu <legendecas@gmail.com>
Reviewed-By: Ethan Arrowood <ethan@arrowood.dev>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: LiviaMedeiros <livia@cirno.name>

Author: dario-piotrowicz

test/parallel/test-source-map-enable.js

@@ -1,7 +1,5 @@
 'use strict';
 
-if (!process.features.inspector) return;
-
 const common = require('../common');
 const assert = require('assert');
 const { dirname } = require('path');
@@ -10,6 +8,8 @@ const path = require('path');
 const { spawnSync } = require('child_process');
 const { pathToFileURL } = require('url');
 
+common.skipIfInspectorDisabled();
+
 const tmpdir = require('../common/tmpdir');
 tmpdir.refresh();
 

test/parallel/test-v8-coverage.js

@@ -1,13 +1,13 @@
 'use strict';
 
-if (!process.features.inspector) return;
-
 const common = require('../common');
 const assert = require('assert');
 const fs = require('fs');
 const path = require('path');
 const { spawnSync } = require('child_process');
 
+common.skipIfInspectorDisabled();
+
 const tmpdir = require('../common/tmpdir');
 tmpdir.refresh();
 

test/parallel/test-v8-stop-coverage.js

@@ -1,14 +1,14 @@
 'use strict';
 
-if (!process.features.inspector) return;
-
-require('../common');
+const common = require('../common');
 const fixtures = require('../common/fixtures');
 const tmpdir = require('../common/tmpdir');
 const assert = require('assert');
 const fs = require('fs');
 const { spawnSync } = require('child_process');
 
+common.skipIfInspectorDisabled();
+
 tmpdir.refresh();
 const intervals = 20;
 

test/parallel/test-v8-take-coverage-noop.js

@@ -1,14 +1,14 @@
 'use strict';
 
-if (!process.features.inspector) return;
-
-require('../common');
+const common = require('../common');
 const fixtures = require('../common/fixtures');
 const tmpdir = require('../common/tmpdir');
 const assert = require('assert');
 const fs = require('fs');
 const { spawnSync } = require('child_process');
 
+common.skipIfInspectorDisabled();
+
 tmpdir.refresh();
 
 // v8.takeCoverage() should be a noop if NODE_V8_COVERAGE is not set.

test/parallel/test-v8-take-coverage.js

@@ -1,14 +1,14 @@
 'use strict';
 
-if (!process.features.inspector) return;
-
-require('../common');
+const common = require('../common');
 const fixtures = require('../common/fixtures');
 const tmpdir = require('../common/tmpdir');
 const assert = require('assert');
 const fs = require('fs');
 const { spawnSync } = require('child_process');
 
+common.skipIfInspectorDisabled();
+
 tmpdir.refresh();
 const intervals = 40;
 // Outputs coverage when v8.takeCoverage() is invoked.