Skip to content

Commit

Permalink
fix: increased maxDataSize for multicall test
Browse files Browse the repository at this point in the history
Signed-off-by: Logan Nguyen <[email protected]>
  • Loading branch information
quiet-node committed Jun 11, 2024
1 parent 6ad1ff3 commit e33230b
Showing 1 changed file with 6 additions and 4 deletions.
10 changes: 6 additions & 4 deletions test/multicall/Multicall.js
Original file line number Diff line number Diff line change
Expand Up @@ -263,9 +263,11 @@ describe('Multicall Test Suite', function () {
expect(bytes).to.gte(42624);
});

it('should NOT be able to aggregate 115 calls to processLongOutput', async function () {
const n = 115;
const maxDataSize = 25 * 1024 * 2; // 25 kb
it('should NOT be able to aggregate 585 calls to processLongOutput', async function () {
// @note: since [email protected], the maximum data size was increased to 128 KiB.
const maxDataSize = 128 * 1024 * 2; // 262144 characters - 128 KiB
const n = 585; // 262218 characters ~ 128.03 KiB

let hasError = false;
try {
await multicallProcessLongOutput(n);
Expand All @@ -275,7 +277,7 @@ describe('Multicall Test Suite', function () {
expect(e.message).to.exist;

// Output is too large and the call is reverted.
// The call exceeded the call size limit of 25KB
// The call exceeded the call size limit of 128 KiB
const EXPECTED_ERROR_MESSAGE = `exceeds ${maxDataSize} characters`;
expect(e.message).to.contain(EXPECTED_ERROR_MESSAGE);
}
Expand Down

0 comments on commit e33230b

Please sign in to comment.