OSSForks/node
1
0
Fork 0
mirror of https://github.com/zebrajr/node.git synced 2025-12-06 00:20:08 +01:00
Code Issues Actions 8 Packages Projects Releases Wiki Activity

test: fix status when compiled without inspector

Browse Source 
PR-URL: https://github.com/nodejs/node/pull/60289
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Ulises Gascón <ulisesgascongonzalez@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Matteo Collina <matteo.collina@gmail.com>
This commit is contained in:
Antoine du Hamel 2025-10-19 19:39:54 +02:00 committed by GitHub
parent 3437e1c4bd
commit f25cce2805
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
5 changed files with 17 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
test/common/index.js
View File

@ -54,6 +54,7 @@ const noop = () => {};
const hasCrypto = Boolean(process.versions.openssl) &&
!process.env.NODE_SKIP_CRYPTO;
const hasInspector = Boolean(process.features.inspector);
const hasSQLite = Boolean(process.versions.sqlite);
const hasQuic = hasCrypto && !!process.features.quic;
@ -711,7 +712,7 @@ function expectsError(validator, exact) {
}
function skipIfInspectorDisabled() {
if (!process.features.inspector) {
if (!hasInspector) {
skip('V8 inspector is disabled');
}
}
@ -930,6 +931,7 @@ const common = {
hasIntl,
hasCrypto,
hasQuic,
hasInspector,
hasSQLite,
invalidArgTypeHelper,
isAlive,

2
test/common/index.mjs
View File

@ -17,6 +17,7 @@ const {
getTTYfd,
hasCrypto,
hasQuic,
hasInspector,
hasSQLite,
hasIntl,
hasIPv6,
@ -68,6 +69,7 @@ export {
getTTYfd,
hasCrypto,
hasQuic,
hasInspector,
hasSQLite,
hasIntl,
hasIPv6,

6
test/common/wpt.js
View File

@ -412,6 +412,9 @@ class BuildRequirement {
// Not using common.hasCrypto because of the global leak checks
this.hasCrypto = Boolean(process.versions.openssl) &&
!process.env.NODE_SKIP_CRYPTO;
// Not using common.hasInspector because of the global leak checks
this.hasInspector = Boolean(process.features.inspector);
}
/**
@ -429,6 +432,9 @@ class BuildRequirement {
if (requires.has('crypto') && !this.hasCrypto) {
return 'crypto';
}
if (requires.has('inspector') && !this.hasInspector) {
return 'inspector';
}
return false;
}
}

6
test/parallel/test-process-get-builtin.mjs
View File

@ -1,4 +1,4 @@
import { hasCrypto, hasIntl, hasSQLite } from '../common/index.mjs';
import { hasCrypto, hasIntl, hasInspector, hasSQLite } from '../common/index.mjs';
import assert from 'node:assert';
import { builtinModules } from 'node:module';
import { isMainThread } from 'node:worker_threads';
@ -39,6 +39,10 @@ if (!hasIntl) {
// TODO(@jasnell): Remove this once node:quic graduates from unflagged.
publicBuiltins.delete('node:quic');
if (!hasInspector) {
publicBuiltins.delete('inspector');
publicBuiltins.delete('inspector/promises');
}
if (!hasSQLite) {
publicBuiltins.delete('node:sqlite');
}

2
test/wpt/status/console.json
View File

@ -1,7 +1,7 @@
{
"idlharness.any.js": {
"note": "https://github.com/nodejs/node/issues/44185",
"requires": ["crypto", "small-icu"]
"requires": ["crypto", "inspector", "small-icu"]
},
"idlharness-shadowrealm.window.js": {
"skip": "ShadowRealm support is not enabled"