Skip to content

Commit

Permalink
fix: typos
Browse files Browse the repository at this point in the history
Signed-off-by: dxsullivan <[email protected]>
  • Loading branch information
dxsullivan committed Jan 4, 2025
1 parent 721acb1 commit d59a1fb
Show file tree
Hide file tree
Showing 2 changed files with 11 additions and 11 deletions.
4 changes: 2 additions & 2 deletions test/core/Safe.Execution.spec.ts
Original file line number Diff line number Diff line change
Expand Up @@ -271,7 +271,7 @@ describe("Safe", () => {
if (hre.network.zksync) {
// This test fails in zksync because of (allegedly) enormous gas cost differences
// a call to useGas(8) costs ~400k gas in evm but ~28m gas in zksync.
// I suspect the gas cost difference to play a role but the zksync docs do not mention any numbers so i cant confirm this:
// I suspect the gas cost difference to play a role but the zksync docs do not mention any numbers so i can't confirm this:
// https://docs.zksync.io/zk-stack/concepts/fee-mechanism
// From zkSync team:
// Update: in-memory node when in standalone mode assumes very high l1 gas price resulting in a very high gas consumption,
Expand Down Expand Up @@ -333,7 +333,7 @@ describe("Safe", () => {
if (hre.network.zksync) {
// This test fails in zksync because of (allegedly) enormous gas cost differences
// a call to useGas(8) costs ~400k gas in evm but ~28m gas in zksync.
// I suspect the gas cost difference to play a role but the zksync docs do not mention any numbers so i cant confirm this:
// I suspect the gas cost difference to play a role but the zksync docs do not mention any numbers so i can't confirm this:
// https://docs.zksync.io/zk-stack/concepts/fee-mechanism
// From zkSync team:
// Update: in-memory node when in standalone mode assumes very high l1 gas price resulting in a very high gas consumption,
Expand Down
18 changes: 9 additions & 9 deletions test/core/Safe.Setup.spec.ts
Original file line number Diff line number Diff line change
Expand Up @@ -231,14 +231,14 @@ describe("Safe", () => {
/* solhint-enable no-inline-assembly */
}
}`;
const testIntializer = await deployContractFromSource(user1, source);
const testIntializerAddress = await testIntializer.getAddress();
const initData = testIntializer.interface.encodeFunctionData("init", ["0x42baddad"]);
const testInitializer = await deployContractFromSource(user1, source);
const testInitializerAddress = await testInitializer.getAddress();
const initData = testInitializer.interface.encodeFunctionData("init", ["0x42baddad"]);
await expect(
template.setup(
[user1.address, user2.address, user3.address],
2,
testIntializerAddress,
testInitializerAddress,
initData,
AddressOne,
AddressZero,
Expand All @@ -247,7 +247,7 @@ describe("Safe", () => {
),
)
.to.emit(template, "SafeSetup")
.withArgs(safeMsgSender, [user1.address, user2.address, user3.address], 2, testIntializerAddress, AddressOne);
.withArgs(safeMsgSender, [user1.address, user2.address, user3.address], 2, testInitializerAddress, AddressOne);
await expect(await template.domainSeparator()).to.be.eq(calculateSafeDomainSeparator(templateAddress, await chainId()));
await expect(await template.getOwners()).to.be.deep.eq([user1.address, user2.address, user3.address]);
await expect(await template.getThreshold()).to.eq(2n);
Expand All @@ -272,14 +272,14 @@ describe("Safe", () => {
require(false, "Computer says nah");
}
}`;
const testIntializer = await deployContractFromSource(user1, source);
const testIntializerAddress = await testIntializer.getAddress();
const initData = testIntializer.interface.encodeFunctionData("init", ["0x42baddad"]);
const testInitializer = await deployContractFromSource(user1, source);
const testInitializerAddress = await testInitializer.getAddress();
const initData = testInitializer.interface.encodeFunctionData("init", ["0x42baddad"]);
await expect(
template.setup(
[user1.address, user2.address, user3.address],
2,
testIntializerAddress,
testInitializerAddress,
initData,
AddressZero,
AddressZero,
Expand Down

0 comments on commit d59a1fb

Please sign in to comment.