[Tests] log when openssl doesn't support cipher

This fixes browserify#37.
kapouer · Jan 3, 2018 · f542dde · f542dde
1 parent f5f17c2
commit f542dde
Showing 1 changed file with 18 additions and 3 deletions.
diff --git a/test/index.js b/test/index.js
@@ -28,7 +28,6 @@ fixtures.valid.rsa.forEach(function (f) {
     priv = Buffer.from(f['private'], 'base64');
   }
 
-  console.log(nCrypto.getHashes());
   (nCrypto.getHashes().indexOf(f.scheme) >= 0 ? test : test.skip)(f.message, function (t) {
     var bSign;
     try {
@@ -82,8 +81,23 @@ fixtures.valid.ec.forEach(function (f) {
   }
 
   (nCrypto.getHashes().indexOf(f.scheme) >= 0 ? test : test.skip)(f.message, function (t) {
-    var nSign = nCrypto.createSign(f.scheme);
-    var bSign = bCrypto.createSign(f.scheme);
+    var nSign;
+    try {
+      nSign = nCrypto.createSign(f.scheme);
+    } catch (e) {
+      console.info('skipping unsupported browserify-sign scheme', f.scheme);
+      t.end();
+      return;
+    }
+
+    var bSign;
+    try {
+      bSign = bCrypto.createSign(f.scheme);
+    } catch (e) {
+      console.info('skipping unsupported node scheme', f.scheme);
+      t.end();
+      return;
+    }
 
     var bSig = bSign.update(message).sign(priv);
     var nSig = nSign.update(message).sign(priv);
@@ -98,6 +112,7 @@ fixtures.valid.ec.forEach(function (f) {
 
     t.end();
   });
+
   if (f.scheme !== 'DSA' && f.scheme.toLowerCase().indexOf('dsa') === -1) {
     test(f.message + ' named rsa through', function (t) {
       var scheme = 'RSA-' + f.scheme.toUpperCase();