Skip to content

Commit

Permalink
Fix gas limit for execTransactionFromModule in SafeL2 tests
Browse files Browse the repository at this point in the history
  • Loading branch information
mmv08 committed Aug 26, 2024
1 parent da63ade commit ef9ff42
Showing 1 changed file with 1 addition and 4 deletions.
5 changes: 1 addition & 4 deletions test/l2/Safe.Execution.spec.ts
Original file line number Diff line number Diff line change
Expand Up @@ -86,10 +86,7 @@ describe("SafeL2", () => {
const user2Safe = safe.connect(user2);
await executeContractCallWithSigners(safe, safe, "enableModule", [user2.address], [user1]);

//Use manual gasLimit because gas estimation fails for this function on zkSync, though transaction executed successfully
await expect(
user2Safe.execTransactionFromModule(mockAddress, 0, "0xbaddad", 0, { gasLimit: hre.network.zksync ? 250_000 : undefined }),
)
await expect(user2Safe.execTransactionFromModule(mockAddress, 0, "0xbaddad", 0))
.to.emit(safe, "SafeModuleTransaction")
.withArgs(user2.address, mockAddress, 0, "0xbaddad", 0)
.to.emit(safe, "ExecutionFromModuleSuccess")
Expand Down

0 comments on commit ef9ff42

Please sign in to comment.