Skip to content

Commit 86fd1fc

Browse files
yitongdingMylesBorins
authored andcommitted
test: add crypto.scrypt test case with different encoding
PR-URL: #23578 Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Anna Henningsen <anna@addaleax.net> Reviewed-By: Tobias Nießen <tniessen@tnie.de> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
1 parent 94c7406 commit 86fd1fc

File tree

1 file changed

+18
-0
lines changed

1 file changed

+18
-0
lines changed

test/parallel/test-crypto-scrypt.js

Lines changed: 18 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -187,6 +187,24 @@ for (const options of toobig) {
187187
}));
188188
}
189189

190+
{
191+
const defaultEncoding = crypto.DEFAULT_ENCODING;
192+
const defaults = { N: 16384, p: 1, r: 8 };
193+
const expected = crypto.scryptSync('pass', 'salt', 1, defaults);
194+
195+
const testEncoding = 'latin1';
196+
crypto.DEFAULT_ENCODING = testEncoding;
197+
const actual = crypto.scryptSync('pass', 'salt', 1);
198+
assert.deepStrictEqual(actual, expected.toString(testEncoding));
199+
200+
crypto.scrypt('pass', 'salt', 1, common.mustCall((err, actual) => {
201+
assert.ifError(err);
202+
assert.deepStrictEqual(actual, expected.toString(testEncoding));
203+
}));
204+
205+
crypto.DEFAULT_ENCODING = defaultEncoding;
206+
}
207+
190208
for (const { args, expected } of badargs) {
191209
common.expectsError(() => crypto.scrypt(...args), expected);
192210
common.expectsError(() => crypto.scryptSync(...args), expected);

0 commit comments

Comments
 (0)