Skip to content

Commit 93e91a4

Browse files
danbevjasnell
authored andcommitted
test: check inspector support in test/inspector
When configuring node --without-ssl or --without-inspector these test will fail. The underlying issue will be: Inspector support is not available with this Node.js build /work/nodejs/node/out/Release/node: bad option: --inspect=0 This commit adds checks to see if inspector support is enabled and if not skips these tests. PR-URL: #13324 Reviewed-By: Gibson Fahnestock <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Eugene Ostroukhov <[email protected]> Reviewed-By: Sam Roberts <[email protected]> Reviewed-By: James M Snell <[email protected]>
1 parent 47c9de9 commit 93e91a4

File tree

2 files changed

+3
-1
lines changed

2 files changed

+3
-1
lines changed

test/inspector/test-inspector-port-zero-cluster.js

+1
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,7 @@
22
'use strict';
33

44
const common = require('../common');
5+
common.skipIfInspectorDisabled();
56
const assert = require('assert');
67
const cluster = require('cluster');
78

test/inspector/test-inspector-port-zero.js

+2-1
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,7 @@
11
'use strict';
22

3-
const { mustCall } = require('../common');
3+
const { mustCall, skipIfInspectorDisabled } = require('../common');
4+
skipIfInspectorDisabled();
45
const assert = require('assert');
56
const { URL } = require('url');
67
const { spawn } = require('child_process');

0 commit comments

Comments
 (0)