Skip to content

Commit

Permalink
test: clarify assert messages in crypto tests
Browse files Browse the repository at this point in the history
PR-URL: nodejs/node#16019
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: Lance Ball <lball@redhat.com>
  • Loading branch information
cpandrews8 authored and addaleax committed Oct 11, 2017
1 parent 7de4934 commit 172afc0
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions test/parallel/test-crypto-sign-verify.js
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,7 @@ const modSize = 1024;
let s1stream = crypto.createSign('SHA1');
s1stream.end('Test123');
s1stream = s1stream.sign(keyPem, 'base64');
assert.strictEqual(s1, s1stream, 'Stream produces same output');
assert.strictEqual(s1, s1stream, `${s1} should equal ${s1stream}`);

const verified = crypto.createVerify('SHA1')
.update('Test')
Expand All @@ -39,7 +39,7 @@ const modSize = 1024;
let s2stream = crypto.createSign('SHA256');
s2stream.end('Test123');
s2stream = s2stream.sign(keyPem, 'latin1');
assert.strictEqual(s2, s2stream, 'Stream produces same output');
assert.strictEqual(s2, s2stream, `${s2} should equal ${s2stream}`);

let verified = crypto.createVerify('SHA256')
.update('Test')
Expand Down

0 comments on commit 172afc0

Please sign in to comment.