Permalink
Browse files

test: improve crypto.setEngine coverage to check for errors

PR-URL: #11143
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Yuta Hiroto <hello@about-hiroppy.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
  • Loading branch information...
seppevs authored and jasnell committed Feb 3, 2017
1 parent ef977cf commit ddbfdba4da651c386fc90894a2f327940acf2bdd
Showing with 18 additions and 0 deletions.
  1. +18 −0 test/parallel/test-crypto-engine.js
@@ -0,0 +1,18 @@
'use strict';
const common = require('../common');
if (!common.hasCrypto) {
common.skip('missing crypto');
return;
}
const assert = require('assert');
const crypto = require('crypto');
assert.throws(function() {
crypto.setEngine(true);
}, /^TypeError: "id" argument should be a string$/);
assert.throws(function() {
crypto.setEngine('/path/to/engine', 'notANumber');
}, /^TypeError: "flags" argument should be a number, if present$/);

0 comments on commit ddbfdba

Please sign in to comment.