Skip to content

Commit

Permalink
Update EmailRecoveryModule constructor args in test
Browse files Browse the repository at this point in the history
  • Loading branch information
JohnGuilding committed Aug 1, 2024
1 parent e5f23f5 commit 76f7fb3
Showing 1 changed file with 19 additions and 4 deletions.
23 changes: 19 additions & 4 deletions test/unit/modules/EmailRecoveryModule/constructor.t.sol
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@ import { ISafe } from "src/interfaces/ISafe.sol";
import { EmailRecoveryModuleBase } from "./EmailRecoveryModuleBase.t.sol";
import { EmailRecoveryModule } from "src/modules/EmailRecoveryModule.sol";

contract EmailRecoveryManager_constructor_Test is EmailRecoveryModuleBase {
contract EmailRecoveryModule_constructor_Test is EmailRecoveryModuleBase {
function setUp() public override {
super.setUp();
}
Expand Down Expand Up @@ -67,7 +67,12 @@ contract EmailRecoveryManager_constructor_Test is EmailRecoveryModuleBase {
)
);
new EmailRecoveryModule(
emailRecoveryManagerAddress, validatorAddress, IERC7579Account.execute.selector
address(verifier),
address(dkimRegistry),
address(emailAuthImpl),
address(emailRecoveryHandler),
validatorAddress,
IERC7579Account.execute.selector
);
}

Expand All @@ -78,7 +83,12 @@ contract EmailRecoveryManager_constructor_Test is EmailRecoveryModuleBase {
)
);
new EmailRecoveryModule(
emailRecoveryManagerAddress, validatorAddress, ISafe.setFallbackHandler.selector
address(verifier),
address(dkimRegistry),
address(emailAuthImpl),
address(emailRecoveryHandler),
validatorAddress,
ISafe.setFallbackHandler.selector
);
}

Expand All @@ -89,7 +99,12 @@ contract EmailRecoveryManager_constructor_Test is EmailRecoveryModuleBase {
)
);
new EmailRecoveryModule(
emailRecoveryManagerAddress, validatorAddress, ISafe.setGuard.selector
address(verifier),
address(dkimRegistry),
address(emailAuthImpl),
address(emailRecoveryHandler),
validatorAddress,
ISafe.setGuard.selector
);
}

Expand Down

0 comments on commit 76f7fb3

Please sign in to comment.