Change Mmr Id type to uint128 in L1 msg sender

HerodotusDev · Jun 3, 2024 · a54202d · a54202d
1 parent 0dd027a
commit a54202d
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/l1/src/L1MessagesSender.sol b/l1/src/L1MessagesSender.sol
@@ -56,7 +56,7 @@ contract L1MessagesSender is Ownable {
     }
 
     /// @param aggregatorId The id of a tree previously created by the aggregators factory
-    function sendPoseidonMMRTreeToL2(uint256 aggregatorId, uint256 mmrId) external payable {
+    function sendPoseidonMMRTreeToL2(uint256 aggregatorId, uint128 mmrId) external payable {
         address existingAggregatorAddr = aggregatorsFactory.aggregatorsById(
             aggregatorId
         );
@@ -98,7 +98,7 @@ contract L1MessagesSender is Ownable {
         bytes32 poseidonMMRRoot,
         uint256 mmrSize,
         uint256 aggregatorId,
-        uint256 mmrId
+        uint128 mmrId
     ) internal {
         uint256[] memory message = new uint256[](4);
 

diff --git a/l1/test/L1MessagesSender.t.sol b/l1/test/L1MessagesSender.t.sol
@@ -35,7 +35,7 @@ contract L1MessagesSenderTest is Test {
         // This aggregator id must exist in the factory
         uint256 aggregatorId = 1;
 
-        uint256 mmrId = 4;
+        uint128 mmrId = 4;
 
         // Value must be greater than 0
         sender.sendPoseidonMMRTreeToL2{value: 1}(aggregatorId, mmrId);