Skip to content

Commit f542dde

Browse files
kapouerljharb
authored andcommitted
[Tests] log when openssl doesn't support cipher
This fixes #37.
1 parent f5f17c2 commit f542dde

File tree

1 file changed

+18
-3
lines changed

1 file changed

+18
-3
lines changed

test/index.js

Lines changed: 18 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,6 @@ fixtures.valid.rsa.forEach(function (f) {
2828
priv = Buffer.from(f['private'], 'base64');
2929
}
3030

31-
console.log(nCrypto.getHashes());
3231
(nCrypto.getHashes().indexOf(f.scheme) >= 0 ? test : test.skip)(f.message, function (t) {
3332
var bSign;
3433
try {
@@ -82,8 +81,23 @@ fixtures.valid.ec.forEach(function (f) {
8281
}
8382

8483
(nCrypto.getHashes().indexOf(f.scheme) >= 0 ? test : test.skip)(f.message, function (t) {
85-
var nSign = nCrypto.createSign(f.scheme);
86-
var bSign = bCrypto.createSign(f.scheme);
84+
var nSign;
85+
try {
86+
nSign = nCrypto.createSign(f.scheme);
87+
} catch (e) {
88+
console.info('skipping unsupported browserify-sign scheme', f.scheme);
89+
t.end();
90+
return;
91+
}
92+
93+
var bSign;
94+
try {
95+
bSign = bCrypto.createSign(f.scheme);
96+
} catch (e) {
97+
console.info('skipping unsupported node scheme', f.scheme);
98+
t.end();
99+
return;
100+
}
87101

88102
var bSig = bSign.update(message).sign(priv);
89103
var nSig = nSign.update(message).sign(priv);
@@ -98,6 +112,7 @@ fixtures.valid.ec.forEach(function (f) {
98112

99113
t.end();
100114
});
115+
101116
if (f.scheme !== 'DSA' && f.scheme.toLowerCase().indexOf('dsa') === -1) {
102117
test(f.message + ' named rsa through', function (t) {
103118
var scheme = 'RSA-' + f.scheme.toUpperCase();

0 commit comments

Comments
 (0)