Skip to content

Commit

Permalink
tests: update test-tls-no-cert-required
Browse files Browse the repository at this point in the history
Change var to const.
Wrap callback of createServer with common.mustCall.
  • Loading branch information
Raja Panidepu committed Dec 1, 2016
1 parent 8264a22 commit d5d7db4
Showing 1 changed file with 6 additions and 5 deletions.
11 changes: 6 additions & 5 deletions test/parallel/test-tls-no-cert-required.js
Original file line number Diff line number Diff line change
@@ -1,15 +1,16 @@
'use strict';
var common = require('../common');
const common = require('../common');

if (!common.hasCrypto) {
common.skip('missing crypto');
return;
}
var tls = require('tls');
const tls = require('tls');

// Omitting the cert or pfx option to tls.createServer() should not throw.
// AECDH-NULL-SHA is a no-authentication/no-encryption cipher and hence
// doesn't need a certificate.
tls.createServer({ ciphers: 'AECDH-NULL-SHA' }).listen(0, function() {
this.close();
});
tls.createServer({ ciphers: 'AECDH-NULL-SHA' })
.listen(0, common.mustCall(function() {
this.close();
}));

0 comments on commit d5d7db4

Please sign in to comment.