Skip to content

Commit 58509ec

Browse files
cpandrews8MylesBorins
authored andcommitted
test: clarify assert messages in crypto tests
PR-URL: #16019 Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de> Reviewed-By: Lance Ball <lball@redhat.com>
1 parent c64603f commit 58509ec

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

test/parallel/test-crypto-sign-verify.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,7 @@ const modSize = 1024;
2727
let s1stream = crypto.createSign('SHA1');
2828
s1stream.end('Test123');
2929
s1stream = s1stream.sign(keyPem, 'base64');
30-
assert.strictEqual(s1, s1stream, 'Stream produces same output');
30+
assert.strictEqual(s1, s1stream, `${s1} should equal ${s1stream}`);
3131

3232
const verified = crypto.createVerify('SHA1')
3333
.update('Test')
@@ -43,7 +43,7 @@ const modSize = 1024;
4343
let s2stream = crypto.createSign('SHA256');
4444
s2stream.end('Test123');
4545
s2stream = s2stream.sign(keyPem, 'latin1');
46-
assert.strictEqual(s2, s2stream, 'Stream produces same output');
46+
assert.strictEqual(s2, s2stream, `${s2} should equal ${s2stream}`);
4747

4848
let verified = crypto.createVerify('SHA256')
4949
.update('Test')

0 commit comments

Comments
 (0)