From 8f78c6646e32a54cbc9fe8b03a80947b6cecb412 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' });