Skip to content

Commit

Permalink
test: add asserts to validate test assumptions
Browse files Browse the repository at this point in the history
Refs: nodejs#54968
Refs: nodejs#53382

Add additional asserts as suggestd by Richard in:
nodejs#54968

Signed-off-by: Michael Dawson <[email protected]>
PR-URL: nodejs#54997
Reviewed-By: Richard Lau <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
  • Loading branch information
mhdawson authored and tpoisseau committed Nov 21, 2024
1 parent 6be5479 commit 0594ed0
Showing 1 changed file with 3 additions and 0 deletions.
3 changes: 3 additions & 0 deletions test/parallel/test-tls-cert-regression.js
Original file line number Diff line number Diff line change
Expand Up @@ -22,6 +22,7 @@
'use strict';
const common = require('../common');
const fixtures = require('../common/fixtures');
const assert = require('assert');

if (!common.hasCrypto)
common.skip('missing crypto');
Expand All @@ -43,6 +44,8 @@ while (cert[cert.length - 1 - i] === 0x0a) i++;
if (i !== 0) cert = cert.slice(0, cert.length - i);

function test(cert, key, cb) {
assert.notStrictEqual(cert.at(-1), 0x0a);
assert.notStrictEqual(key.at(-1), 0x0a);
const server = tls.createServer({
cert,
key
Expand Down

0 comments on commit 0594ed0

Please sign in to comment.