From 649288bef63e89186455f69d0a9e9690cd98c53e Mon Sep 17 00:00:00 2001 From: Hariss096 Date: Thu, 6 Sep 2018 03:03:04 +0500 Subject: [PATCH] test: remove string argument to strictEqual() Removing string literal passed as argument to assert.strictEqual() in order to print the expected value instead of the string. (First issue assigned through Email by Rich Trott) PR-URL: https://github.com/nodejs/node/pull/22718 Reviewed-By: Colin Ihrig Reviewed-By: Trivikram Kamat Reviewed-By: Rich Trott Reviewed-By: Luigi Pinca Reviewed-By: James M Snell --- test/sequential/test-crypto-timing-safe-equal.js | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/test/sequential/test-crypto-timing-safe-equal.js b/test/sequential/test-crypto-timing-safe-equal.js index 6aaf7de2284a67..dcebef29d782b0 100644 --- a/test/sequential/test-crypto-timing-safe-equal.js +++ b/test/sequential/test-crypto-timing-safe-equal.js @@ -6,16 +6,16 @@ if (!common.hasCrypto) const assert = require('assert'); const crypto = require('crypto'); +// 'should consider equal strings to be equal' assert.strictEqual( crypto.timingSafeEqual(Buffer.from('foo'), Buffer.from('foo')), - true, - 'should consider equal strings to be equal' + true ); +// 'should consider unequal strings to be unequal' assert.strictEqual( crypto.timingSafeEqual(Buffer.from('foo'), Buffer.from('bar')), - false, - 'should consider unequal strings to be unequal' + false ); common.expectsError(