move icon indicating copy to clipboard operation
move copied to clipboard

[evm] reverts with "transaction ran out of gas"

Open junkil-park opened this issue 3 years ago • 0 comments

the issue with "transaction ran out of gas" appears again.

this time, array arguments in external calls are suspicious.

See the following in ExternalCall.test.js:

        // TODO: Error with "Transaction ran out of gas"
        // it('receiver reverting with error message', async function () {
        //     const ids = [1,2,3];
        //     const amounts = [10, 100, 1000];
        //     const data = '0x42';
        //     const receiver = await ERC1155Receiver.new(RECEIVER_SINGLE_MAGIC_VALUE, false, RECEIVER_BATCH_MAGIC_VALUE, true);
        //     await expectRevert(
        //         this.externalCall.doSafeBatchTransferAcceptanceCheck(this.externalCall.address, this.externalCall.address, receiver.address, ids, amounts, data),
        //         'err_reason',
        //     );
        // });
        // TODO: Error with "Transaction ran out of gas"
        // it('receiver reverting without error message', async function () {
        //     const ids = [1,2,3];
        //     const amounts = [10, 100, 1000];
        //     const data = '0x42';
        //     const receiver = await ERC721Receiver.new(RECEIVER_MAGIC_VALUE, Error.Panic);
        //     await expectRevert(
        //         this.externalCall.doSafeBatchTransferAcceptanceCheck(this.externalCall.address, this.externalCall.address, receiver.address, ids, amounts, data),
        //         'err_data',
        //     );
        // });

junkil-park avatar Apr 17 '22 15:04 junkil-park