test: improve crypto coverage

* call Certificate function directly * check exception when sign option is undefined PR-URL: #11280 Reviewed-By: Rich Trott <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Santiago Gimeno <[email protected]> Reviewed-By: James M Snell <[email protected]>
nodejs · Feb 22, 2017 · 348f2ef · 348f2ef
1 parent 4d7c942
commit 348f2ef
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 0 deletions.
diff --git a/test/parallel/test-crypto-certificate.js b/test/parallel/test-crypto-certificate.js
@@ -37,3 +37,6 @@ assert.strictEqual(certificate.exportChallenge(spkacFail), '');
 function stripLineEndings(obj) {
   return obj.replace(/\n/g, '');
 }
+
+// direct call Certificate() should return instance
+assert(crypto.Certificate() instanceof crypto.Certificate);
diff --git a/test/parallel/test-crypto-sign-verify.js b/test/parallel/test-crypto-sign-verify.js
@@ -70,3 +70,10 @@ const keyPem = fs.readFileSync(common.fixturesDir + '/test_key.pem', 'ascii');
   verified = verStream.verify(certPem, s3);
   assert.strictEqual(verified, true, 'sign and verify (stream)');
 }
+
+// Test throws exception when key options is null
+{
+  assert.throws(() => {
+    crypto.createSign('RSA-SHA1').update('Test123').sign(null, 'base64');
+  }, /^Error: No key provided to sign$/);
+}