Skip to content

Commit

Permalink
Browse files Browse the repository at this point in the history
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: #38560
Reviewed-By: Darshan Sen <raisinten@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Zijian Liu <lxxyxzj@gmail.com>
  • Loading branch information
VoltrexKeyva authored and targos committed Jun 11, 2021
1 parent c5b86f8 commit d7c6a3e
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions test/js-native-api/test_bigint/test.js
Expand Up @@ -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'
});

0 comments on commit d7c6a3e

Please sign in to comment.