Add paginated method to get all accounts: getAccounts and totalAccounts (#564)

Add getAccounts and totalAccounts

Author: nkrishang

contracts/prebuilts/account/utils/BaseAccountFactory.sol

@@ -112,6 +112,27 @@ abstract contract BaseAccountFactory is IAccountFactory, Multicall {
         return _baseAccountFactoryStorage().allAccounts.contains(_account);
     }
 
+    /// @notice Returns the total number of accounts.
+    function totalAccounts() external view returns (uint256) {
+        return _baseAccountFactoryStorage().allAccounts.length();
+    }
+
+    /// @notice Returns all accounts between the given indices.
+    function getAccounts(uint256 _start, uint256 _end) external view returns (address[] memory accounts) {
+        uint256 len = _baseAccountFactoryStorage().allAccounts.length();
+        require(_start < _end && _end <= len, "BaseAccountFactory: invalid indices");
+
+        if (len == 0) {
+            return new address[](0);
+        }
+
+        accounts = new address[](_end - _start);
+
+        for (uint256 i = _start; i < _end; i += 1) {
+            accounts[i] = _baseAccountFactoryStorage().allAccounts.at(i);
+        }
+    }
+
     /// @notice Returns all accounts created on the factory.
     function getAllAccounts() external view returns (address[] memory) {
         return _baseAccountFactoryStorage().allAccounts.values();

src/test/smart-wallet/Account.t.sol

@@ -353,6 +353,7 @@ contract SimpleAccountTest is BaseTest {
 
         address[] memory allAccounts = accountFactory.getAllAccounts();
         assertEq(allAccounts.length, amount);
+        assertEq(accountFactory.totalAccounts(), amount);
 
         for (uint256 i = 0; i < amount; i += 1) {
             assertEq(