Skip to content

Commit 688bdfe

Browse files
danbevtargos
authored andcommitted
test: add crypto check to test-http2-debug
This commit adds a crypto check to test-http2-debug.js as it currently will error if configured --without-ssl. The issue here is that the while the test spawns a child process that runs test-http2-ping.js, which does have a crypto check, it will just print '1..0 # Skipped: missing crypto' to stdout, and nothing to stderr which is what this test is trying to assert and hence failing. PR-URL: #21205 Reviewed-By: Daijiro Wachi <[email protected]> Reviewed-By: Ujjwal Sharma <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Richard Lau <[email protected]> Reviewed-By: James M Snell <[email protected]> Reviewed-By: Luigi Pinca <[email protected]>
1 parent 6aade4a commit 688bdfe

File tree

1 file changed

+3
-1
lines changed

1 file changed

+3
-1
lines changed

test/parallel/test-http2-debug.js

+3-1
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,7 @@
11
'use strict';
2-
require('../common');
2+
const common = require('../common');
3+
if (!common.hasCrypto)
4+
common.skip('missing crypto');
35
const assert = require('assert');
46
const child_process = require('child_process');
57
const path = require('path');

0 commit comments

Comments
 (0)