From d7c6a3eb03f4108c0aa538c424593a6914c93d3d Mon Sep 17 00:00:00 2001 From: Voltrex <62040526+VoltrexMaster@users.noreply.github.com> Date: Wed, 5 May 2021 20:12:32 +0430 Subject: [PATCH] test: call functions internally All (or at least most) of the tests uses lambdas (or arrow functions if you will) to call these functions internally inside of directly calling them, this should also use this technique for consistency. PR-URL: https://github.com/nodejs/node/pull/38560 Reviewed-By: Darshan Sen Reviewed-By: James M Snell Reviewed-By: Zijian Liu --- test/js-native-api/test_bigint/test.js | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/test/js-native-api/test_bigint/test.js b/test/js-native-api/test_bigint/test.js index bf9ce5066d6d2a..509c263c733a53 100644 --- a/test/js-native-api/test_bigint/test.js +++ b/test/js-native-api/test_bigint/test.js @@ -40,13 +40,13 @@ const { assert.strictEqual(num, TestWords(num)); }); -assert.throws(CreateTooBigBigInt, { +assert.throws(() => CreateTooBigBigInt(), { name: 'Error', message: 'Invalid argument', }); // Test that we correctly forward exceptions from the engine. -assert.throws(MakeBigIntWordsThrow, { +assert.throws(() => MakeBigIntWordsThrow(), { name: 'RangeError', message: 'Maximum BigInt size exceeded' });