chore: move userOpMaxCost to bottom and remove initialisation on init…

…ialize
KintoXYZ · Jan 30, 2024 · ba2a561 · ba2a561
1 parent 2e9fb4c
commit ba2a561
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 2 deletions.
diff --git a/src/paymasters/SponsorPaymaster.sol b/src/paymasters/SponsorPaymaster.sol
@@ -31,7 +31,6 @@ contract SponsorPaymaster is Initializable, BasePaymaster, UUPSUpgradeable, Reen
     uint256 public constant RATE_LIMIT_PERIOD = 1 minutes;
     uint256 public constant RATE_LIMIT_THRESHOLD_TOTAL = 50;
 
-    uint256 public userOpMaxCost;
     IKintoAppRegistry public override appRegistry;
 
     mapping(address => uint256) public balances;
@@ -45,6 +44,8 @@ contract SponsorPaymaster is Initializable, BasePaymaster, UUPSUpgradeable, Reen
     // rate limit across apps: user => RateLimitData
     mapping(address => ISponsorPaymaster.RateLimitData) public globalRateLimit;
 
+    uint256 public userOpMaxCost;
+
     // ========== Events ============
 
     event AppRegistrySet(address oldRegistry, address newRegistry);
@@ -65,7 +66,6 @@ contract SponsorPaymaster is Initializable, BasePaymaster, UUPSUpgradeable, Reen
         __UUPSUpgradeable_init();
         _transferOwnership(_owner);
 
-        userOpMaxCost = 0.03 ether;
         // unlocks owner
         unlockBlock[_owner] = block.number;
     }

diff --git a/test/helpers/AATestScaffolding.sol b/test/helpers/AATestScaffolding.sol
@@ -150,6 +150,9 @@ abstract contract AATestScaffolding is KYCSignature {
         // Set the registry in the paymaster
         _paymaster.setAppRegistry(address(_kintoAppRegistry));
 
+        // Set user op max cost
+        _paymaster.setUserOpMaxCost(0.03 ether);
+
         vm.stopPrank();
     }