test: use common.skipIfInspectorDisable() to skip tests

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

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.