Skip to content

Commit 18910fd

Browse files
yitongdingBridgeAR
authored andcommitted
test: add crypto.scrypt test case with different encoding
PR-URL: nodejs#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 d0d6ba0 commit 18910fd

File tree

1 file changed

+18
-0
lines changed

1 file changed

+18
-0
lines changed

test/parallel/test-crypto-scrypt.js

+18
Original file line numberDiff line numberDiff line change
@@ -189,6 +189,24 @@ for (const options of toobig) {
189189
}));
190190
}
191191

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

0 commit comments

Comments
 (0)