update base contract

test/base/SpendPermissionsBase.sol

@@ -14,16 +14,21 @@ contract SpendPermissionsBase {
         mockSpendPermissions = new MockSpendPermissions();
     }
 
-    function _createRecurringAllowance(uint48 start, uint48 period, uint160 allowance)
-        internal
-        pure
-        returns (SpendPermissions.RecurringAllowance memory)
-    {
-        return SpendPermissions.RecurringAllowance(start, period, allowance);
+    function _createRecurringAllowance(
+        address account,
+        address spender,
+        address token,
+        uint48 start,
+        uint48 end,
+        uint48 period,
+        uint160 allowance
+    ) internal pure returns (SpendPermissions.RecurringAllowance memory) {
+        return SpendPermissions.RecurringAllowance(account, spender, token, start, end, period, allowance);
     }
 
+    // we'll keep this around until we know we do or don't need it
     function _safeAdd(uint48 a, uint48 b) internal pure returns (uint48 c) {
         bool overflow = uint256(a) + uint256(b) > type(uint48).max;
         return overflow ? type(uint48).max : a + b;
     }
-}
+}

test/mocks/MockSpendPermissions.sol

@@ -4,9 +4,8 @@ pragma solidity ^0.8.23;
 import {SpendPermissions} from "../../src/SpendPermissions.sol";
 
 contract MockSpendPermissions is SpendPermissions {
-
     // TODO: what other internal functions should be exposed?
-    
+
     // function initializeRecurringAllowance(
     //     address account,
     //     bytes32 permissionHash,
@@ -18,4 +17,4 @@ contract MockSpendPermissions is SpendPermissions {
     function useRecurringAllowance(RecurringAllowance memory recurringAllowance, uint256 value) public {
         _useRecurringAllowance(recurringAllowance, value);
     }
-}
+}