Add test cases for EnumerableGuardianMap.

src/libraries/EnumerableGuardianMap.sol

@@ -68,7 +68,7 @@ library EnumerableGuardianMap {
         returns (bool)
     {
         uint256 length = map._keys.length();
-        if (length > MAX_NUMBER_OF_GUARDIANS) {
+        if (length >= MAX_NUMBER_OF_GUARDIANS) {
             revert MaxNumberOfGuardiansReached();
         }
         map._values[key] = value;

test/integration/IntegrationBase.t.sol

@@ -23,6 +23,7 @@ abstract contract IntegrationBase is RhinestoneModuleKit, Test {
     AccountInstance instance;
     address accountAddress;
     address owner;
+    address anotherOwner;
     address newOwner;
 
     // recovery config
@@ -66,9 +67,11 @@ abstract contract IntegrationBase is RhinestoneModuleKit, Test {
 
         // create owners
         owner = vm.createWallet("owner").addr;
+        anotherOwner = vm.createWallet("anotherOwner").addr;
         newOwner = vm.createWallet("newOwner").addr;
-        address[] memory owners = new address[](1);
+        address[] memory owners = new address[](2);
         owners[0] = owner;
+        owners[1] = anotherOwner;
 
         // Deploy and fund the account
         instance = makeAccountInstance("account");

test/unit/EmailRecoveryFactory/deployModuleAndManager.t.sol

@@ -3,13 +3,20 @@ pragma solidity ^0.8.25;
 
 import "forge-std/console2.sol";
 import { UnitBase } from "../UnitBase.t.sol";
+import { EmailRecoveryFactory } from "../../../src/EmailRecoveryFactory.sol";
 
-contract EmailRecoveryManager_deployModuleAndManager_Test is UnitBase {
+contract EmailRecoveryFactory_deployModuleAndManager_Test is UnitBase {
     function setUp() public override {
         super.setUp();
+        emailRecoveryFactory = new EmailRecoveryFactory();
     }
 
     function test_DeployModuleAndManager_Succeeds() public {
-        // TODO: test
+        emailRecoveryFactory.deployModuleAndManager(
+            address(verifier),
+            address(dkimRegistry),
+            address(emailAuthImpl),
+            address(emailRecoveryHandler)
+        );
     }
 }

test/unit/EmailRecoveryManager/getGuardian.t.sol

@@ -2,17 +2,30 @@
 pragma solidity ^0.8.25;
 
 import "forge-std/console2.sol";
-import { IEmailRecoveryManager } from "src/interfaces/IEmailRecoveryManager.sol";
-import { EmailRecoveryModule } from "src/modules/EmailRecoveryModule.sol";
-import { GuardianStorage, GuardianStatus } from "src/libraries/EnumerableGuardianMap.sol";
-import { UnitBase } from "../UnitBase.t.sol";
+import {IEmailRecoveryManager} from "src/interfaces/IEmailRecoveryManager.sol";
+import {EmailRecoveryModule} from "src/modules/EmailRecoveryModule.sol";
+import {GuardianStorage, GuardianStatus} from "src/libraries/EnumerableGuardianMap.sol";
+import {UnitBase} from "../UnitBase.t.sol";
 
 contract EmailRecoveryManager_getGuardian_Test is UnitBase {
+    address newGuardian = address(1);
+    uint256 newGuardianWeight = 1;
+
     function setUp() public override {
         super.setUp();
+
+        vm.startPrank(accountAddress);
+        emailRecoveryManager.addGuardian(newGuardian, newGuardianWeight);
+        vm.stopPrank();
     }
 
     function test_GetGuardian_Succeeds() public {
-        // TODO: test
+        GuardianStorage memory guardianStorage = emailRecoveryManager
+            .getGuardian(accountAddress, newGuardian);
+        assertEq(
+            uint256(guardianStorage.status),
+            uint256(GuardianStatus.REQUESTED)
+        );
+        assertEq(guardianStorage.weight, newGuardianWeight);
     }
 }

test/unit/EmailRecoveryManager/getGuardianConfig.t.sol

@@ -2,17 +2,39 @@
 pragma solidity ^0.8.25;
 
 import "forge-std/console2.sol";
-import { IEmailRecoveryManager } from "src/interfaces/IEmailRecoveryManager.sol";
-import { EmailRecoveryModule } from "src/modules/EmailRecoveryModule.sol";
-import { GuardianStorage, GuardianStatus } from "src/libraries/EnumerableGuardianMap.sol";
-import { UnitBase } from "../UnitBase.t.sol";
+import {IEmailRecoveryManager} from "src/interfaces/IEmailRecoveryManager.sol";
+import {EmailRecoveryModule} from "src/modules/EmailRecoveryModule.sol";
+import {GuardianStorage, GuardianStatus} from "src/libraries/EnumerableGuardianMap.sol";
+import {UnitBase} from "../UnitBase.t.sol";
 
 contract EmailRecoveryManager_getGuardianConfig_Test is UnitBase {
+    address newGuardian = address(1);
+    uint256 newGuardianWeight = 1;
+
+    uint256 expectedGuardianCount;
+    uint256 expectedTotalWeight;
+    uint256 expectedThreshold;
+
     function setUp() public override {
         super.setUp();
+
+        expectedGuardianCount = guardians.length + 1;
+        expectedTotalWeight = totalWeight + newGuardianWeight;
+        expectedThreshold = threshold;
+
+        vm.startPrank(accountAddress);
+        emailRecoveryManager.addGuardian(newGuardian, newGuardianWeight);
+        vm.stopPrank();
     }
 
     function test_GetGuardianConfig_Succeeds() public {
-        // TODO: test
+        IEmailRecoveryManager.GuardianConfig
+            memory guardianConfig = emailRecoveryManager.getGuardianConfig(
+                accountAddress
+            );
+        console2.log(expectedGuardianCount);
+        assertEq(guardianConfig.guardianCount, expectedGuardianCount);
+        assertEq(guardianConfig.totalWeight, expectedTotalWeight);
+        assertEq(guardianConfig.threshold, expectedThreshold);
     }
 }

test/unit/EmailRecoveryManager/getRecoveryConfig.t.sol

@@ -2,17 +2,38 @@
 pragma solidity ^0.8.25;
 
 import "forge-std/console2.sol";
-import { IEmailRecoveryManager } from "src/interfaces/IEmailRecoveryManager.sol";
-import { EmailRecoveryModule } from "src/modules/EmailRecoveryModule.sol";
-import { GuardianStorage, GuardianStatus } from "src/libraries/EnumerableGuardianMap.sol";
-import { UnitBase } from "../UnitBase.t.sol";
+import {IEmailRecoveryManager} from "src/interfaces/IEmailRecoveryManager.sol";
+import {EmailRecoveryModule} from "src/modules/EmailRecoveryModule.sol";
+import {GuardianStorage, GuardianStatus} from "src/libraries/EnumerableGuardianMap.sol";
+import {UnitBase} from "../UnitBase.t.sol";
 
 contract EmailRecoveryManager_getRecoveryConfig_Test is UnitBase {
+    uint256 newDelay = 1 days;
+    uint256 newExpiry = 4 weeks;
+
     function setUp() public override {
         super.setUp();
+
+        IEmailRecoveryManager.RecoveryConfig
+            memory recoveryConfig = IEmailRecoveryManager.RecoveryConfig(
+                newDelay,
+                newExpiry
+            );
+
+        vm.startPrank(accountAddress);
+        emailRecoveryManager.updateRecoveryConfig(recoveryConfig);
+
+        recoveryConfig = emailRecoveryManager.getRecoveryConfig(accountAddress);
+        assertEq(recoveryConfig.delay, newDelay);
+        assertEq(recoveryConfig.expiry, newExpiry);
     }
 
     function test_GetRecoveryConfig_Succeeds() public {
-        // TODO: test
+        IEmailRecoveryManager.RecoveryConfig
+            memory result = emailRecoveryManager.getRecoveryConfig(
+                accountAddress
+            );
+        assertEq(result.delay, newDelay);
+        assertEq(result.expiry, newExpiry);
     }
 }

test/unit/EmailRecoveryManager/getRecoveryRequest.t.sol

@@ -2,17 +2,29 @@
 pragma solidity ^0.8.25;
 
 import "forge-std/console2.sol";
-import { IEmailRecoveryManager } from "src/interfaces/IEmailRecoveryManager.sol";
-import { EmailRecoveryModule } from "src/modules/EmailRecoveryModule.sol";
-import { GuardianStorage, GuardianStatus } from "src/libraries/EnumerableGuardianMap.sol";
-import { UnitBase } from "../UnitBase.t.sol";
+import {IEmailRecoveryManager} from "src/interfaces/IEmailRecoveryManager.sol";
+import {EmailRecoveryModule} from "src/modules/EmailRecoveryModule.sol";
+import {GuardianStorage, GuardianStatus} from "src/libraries/EnumerableGuardianMap.sol";
+import {UnitBase} from "../UnitBase.t.sol";
+import {Strings} from "@openzeppelin/contracts/utils/Strings.sol";
 
 contract EmailRecoveryManager_getRecoveryRequest_Test is UnitBase {
     function setUp() public override {
         super.setUp();
     }
 
     function test_GetRecoveryRequest_Succeeds() public {
-        // TODO: test
+        acceptGuardian(accountSalt1);
+        vm.warp(12 seconds);
+        handleRecovery(recoveryModuleAddress, calldataHash, accountSalt1);
+
+        IEmailRecoveryManager.RecoveryRequest
+            memory recoveryRequest = emailRecoveryManager.getRecoveryRequest(
+                accountAddress
+            );
+        assertEq(recoveryRequest.executeAfter, 0);
+        assertEq(recoveryRequest.executeBefore, 0);
+        assertEq(recoveryRequest.currentWeight, 1);
+        assertEq(recoveryRequest.calldataHashString, "");
     }
 }

test/unit/handlers/SafeRecoverySubjectHandler/getPreviousOwnerInLinkedList.t.sol

@@ -47,6 +47,13 @@ contract SafeRecoverySubjectHandler_getPreviousOwnerInLinkedList_Test is SafeUni
     }
 
     function test_GetPreviousOwnerInLinkedList_SucceedsWithMultipleAccounts() public {
-        // TODO: test
+        address expectedPreviousOwner = address(1);
+        address previousOwner =
+            safeRecoverySubjectHandler.exposed_getPreviousOwnerInLinkedList(accountAddress, owner);
+
+        assertEq(expectedPreviousOwner, previousOwner);
+        previousOwner =
+            safeRecoverySubjectHandler.exposed_getPreviousOwnerInLinkedList(accountAddress, anotherOwner);
+        assertEq(expectedPreviousOwner, previousOwner);
     }
 }

test/unit/libraries/EnumerableGuardianMap/get.t.sol

@@ -2,17 +2,42 @@
 pragma solidity ^0.8.25;
 
 import "forge-std/console2.sol";
-import { UnitBase } from "../../UnitBase.t.sol";
+import {UnitBase} from "../../UnitBase.t.sol";
+import {EnumerableGuardianMap, GuardianStorage, GuardianStatus} from "../../../../src/libraries/EnumerableGuardianMap.sol";
 
 contract EnumerableGuardianMap_get_Test is UnitBase {
+    using EnumerableGuardianMap for EnumerableGuardianMap.AddressToGuardianMap;
+    mapping(address account => EnumerableGuardianMap.AddressToGuardianMap guardian)
+        internal guardiansStorage;
+
     function setUp() public override {
         super.setUp();
+        guardiansStorage[accountAddress].set({
+            key: guardian1,
+            value: GuardianStorage(GuardianStatus.REQUESTED, guardianWeights[0])
+        });
     }
 
     function test_Get_GetsExistingValue() public view {
-        // TODO: test
+        GuardianStorage memory result = guardiansStorage[accountAddress].get(
+            guardian1
+        );
+        require(
+            result.status == GuardianStatus.REQUESTED,
+            "Expected status to be REQUESTED"
+        );
+        require(result.weight == guardianWeights[0], "Expected weight to be 1");
     }
+
     function test_Get_GetsNonExistentValue() public view {
-        // TODO: test
+        // It will returns the default value
+        GuardianStorage memory result = guardiansStorage[accountAddress].get(
+            guardian2
+        );
+        require(
+            result.status == GuardianStatus.NONE,
+            "Expected status to be NONE"
+        );
+        require(result.weight == 0, "Expected weight to be 0");
     }
 }

test/unit/libraries/EnumerableGuardianMap/keys.t.sol

@@ -2,23 +2,72 @@
 pragma solidity ^0.8.25;
 
 import "forge-std/console2.sol";
-import { UnitBase } from "../../UnitBase.t.sol";
+import {UnitBase} from "../../UnitBase.t.sol";
+import {EnumerableGuardianMap, GuardianStorage, GuardianStatus} from "../../../../src/libraries/EnumerableGuardianMap.sol";
 
 contract EnumerableGuardianMap_keys_Test is UnitBase {
+    using EnumerableGuardianMap for EnumerableGuardianMap.AddressToGuardianMap;
+    mapping(address account => EnumerableGuardianMap.AddressToGuardianMap guardian)
+        internal guardiansStorage;
+
     function setUp() public override {
         super.setUp();
     }
 
     function test_Keys_StartsEmpty() public view {
-        // TODO: test
+        address[] memory keys = guardiansStorage[accountAddress].keys();
+        assertEq(keys.length, 0);
     }
+
     function test_Keys_ReturnsEmptyArrayOfKeys() public view {
-        // TODO: test
+        // It's same as test_Keys_StartsEmpty()
     }
-    function test_Keys_ReturnsArrayOfKeys() public view {
-        // TODO: test
+
+    function test_Keys_ReturnsArrayOfKeys() public {
+        bool result;
+
+        for (uint256 i = 1; i <= 3; i++) {
+            result = guardiansStorage[accountAddress].set({
+                key: vm.addr(i),
+                value: GuardianStorage(
+                    GuardianStatus.REQUESTED,
+                    guardianWeights[1]
+                )
+            });
+            assertEq(result, true);
+        }
+        address[] memory keys = guardiansStorage[accountAddress].keys();
+        assertEq(keys.length, 3);
+        for (uint256 i = 0; i < 3; i++) {
+            assertEq(keys[i], vm.addr(i + 1));
+        }
     }
-    function test_Keys_ReturnMaxArrayOfKeys() public view {
-        // TODO: test
+
+    function test_Keys_ReturnMaxArrayOfKeys() public {
+        bool result;
+
+        for (
+            uint256 i = 1;
+            i <= EnumerableGuardianMap.MAX_NUMBER_OF_GUARDIANS;
+            i++
+        ) {
+            result = guardiansStorage[accountAddress].set({
+                key: vm.addr(i),
+                value: GuardianStorage(
+                    GuardianStatus.REQUESTED,
+                    guardianWeights[1]
+                )
+            });
+            assertEq(result, true);
+        }
+        address[] memory keys = guardiansStorage[accountAddress].keys();
+        assertEq(keys.length, EnumerableGuardianMap.MAX_NUMBER_OF_GUARDIANS);
+        for (
+            uint256 i = 0;
+            i < EnumerableGuardianMap.MAX_NUMBER_OF_GUARDIANS;
+            i++
+        ) {
+            assertEq(keys[i], vm.addr(i + 1));
+        }
     }
 }