Skip to content

Commit

Permalink
test: refactor parallel/test-tls-cert-chains-concat
Browse files Browse the repository at this point in the history
added common.mustCall and replaced function with arrow function

Fixes : nodejs#14544
  • Loading branch information
juggernaut451 committed Mar 7, 2018
1 parent 9e9c516 commit 8a11196
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions test/parallel/test-tls-cert-chains-concat.js
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
'use strict';
require('../common');
const common = require('../common');
const fixtures = require('../common/fixtures');

// Check cert chain is received by client, and is completed with the ca cert
Expand All @@ -19,7 +19,7 @@ connect({
cert: keys.agent6.cert,
key: keys.agent6.key,
},
}, function(err, pair, cleanup) {
}, common.mustCall((err, pair, cleanup) => {
assert.ifError(err);

const peer = pair.client.conn.getPeerCertificate();
Expand Down Expand Up @@ -47,4 +47,4 @@ connect({
assert.deepStrictEqual(pair.server.conn.getPeerCertificate(true), {});

return cleanup();
});
}));

0 comments on commit 8a11196

Please sign in to comment.