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.

PR-URL: #19096
Fixes: #14544
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Sakthipriyan Vairamani <thechargingvolcano@gmail.com>
  • Loading branch information
juggernaut451 authored and BridgeAR committed Apr 9, 2018
1 parent f3e107a commit cbc7eb7
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 cbc7eb7

Please sign in to comment.