Browse Source

test: fix test-internal-util-assertCrypto regex

When building --without-ssl and then running:

out/Release/node --expose-internals
test/parallel/test-internal-util-assertCrypto.js

The following error is displayed:
assert.js:420
    throw actual;
    ^

Error: Node.js is not compiled with openssl crypto support
    at Object.exports.assertCrypto (internal/util.js:82:11)
    at assert.throws
(/Users/danielbevenius/work/nodejs/node/test/parallel/test-internal-util-assertCrypto.js:8:28)
    at _tryBlock (assert.js:379:5)
    at _throws (assert.js:398:12)
    at Function.throws (assert.js:428:3)
    at Object.<anonymous>
(/Users/danielbevenius/work/nodejs/node/test/parallel/test-internal-util-assertCrypto.js:8:10)
    at Module._compile (module.js:571:32)
    at Object.Module._extensions..js (module.js:580:10)
    at Module.load (module.js:488:32)
    at tryModuleLoad (module.js:447:12)

It looks like the regex is not taking into account the 'Error: ' hence
the failure to match it.

PR-URL: https://github.com/nodejs/node/pull/11620
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Yuta Hiroto <hello@about-hiroppy.com>
Reviewed-By: Santiago Gimeno <santiago.gimeno@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
v7.x
Daniel Bevenius 8 years ago
committed by Anna Henningsen
parent
commit
924b785d50
No known key found for this signature in database GPG Key ID: D8B9F5AEAE84E4CF
  1. 2
      test/parallel/test-internal-util-assertCrypto.js

2
test/parallel/test-internal-util-assertCrypto.js

@ -6,7 +6,7 @@ const util = require('internal/util');
if (!process.versions.openssl) { if (!process.versions.openssl) {
assert.throws(() => util.assertCrypto(), assert.throws(() => util.assertCrypto(),
/^Node.js is not compiled with openssl crypto support$/); /^Error: Node.js is not compiled with openssl crypto support$/);
} else { } else {
assert.doesNotThrow(() => util.assertCrypto()); assert.doesNotThrow(() => util.assertCrypto());
} }

Loading…
Cancel
Save