mirror of
https://github.com/nodejs/node.git
synced 2026-10-14 10:26:26 -04:00
test: skip strace test with shared openssl
`parallel/test-strace-openat-openssl` was added to check explicitly
for a list of known files that would be opened for a set workload
(`require("crypto")`). This is not reliable when Node.js is linked
to an external/shared OpenSSL library (e.g. it might be configured
to load configuration files from a different default location and/or
load more than one configuration file) so skip this test when Node.js
is built in that way.
PR-URL: https://github.com/nodejs/node/pull/61987
Fixes: https://github.com/nodejs/node/issues/61966
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Rafael Gonzaga <rafael.nunu@hotmail.com>
This commit is contained in:
1 parent
7cf2dad715
commit
db278835bc
1 file changed
+3
@@ -11,6 +11,9 @@ if (!common.isLinux)
|
||||
common.skip('linux only');
|
||||
if (common.isASan)
|
||||
common.skip('strace does not work well with address sanitizer builds');
|
||||
if (process.config.variables.node_shared_openssl) {
|
||||
common.skip('external shared openssl may open other files');
|
||||
}
|
||||
if (spawnSync('strace').error !== undefined) {
|
||||
common.skip('missing strace');
|
||||
}
|
||||
|
||||
Reference in new issue
Block a user