fix: remove errors expected value

lidofinance · Nov 6, 2024 · c44a8df · c44a8df
1 parent 071f033
commit c44a8df
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 20 deletions.
diff --git a/contracts/libraries/AssetsAccounting.sol b/contracts/libraries/AssetsAccounting.sol
@@ -122,9 +122,9 @@ library AssetsAccounting {
 
     error InvalidSharesValue(SharesValue value);
     error InvalidUnstETHStatus(uint256 unstETHId, UnstETHRecordStatus status);
-    error InvalidUnstETHHolder(uint256 unstETHId, address actual, address expected);
+    error InvalidUnstETHHolder(uint256 unstETHId, address actual);
     error MinAssetsLockDurationNotPassed(Timestamp lockDurationExpiresAt);
-    error InvalidClaimableAmount(uint256 unstETHId, ETHValue expected, ETHValue actual);
+    error InvalidClaimableAmount(uint256 unstETHId, ETHValue actual);
 
     // ---
     // stETH shares operations accounting
@@ -339,7 +339,7 @@ library AssetsAccounting {
         UnstETHRecord storage unstETHRecord = self.unstETHRecords[unstETHId];
 
         if (unstETHRecord.lockedBy != holder) {
-            revert InvalidUnstETHHolder(unstETHId, holder, unstETHRecord.lockedBy);
+            revert InvalidUnstETHHolder(unstETHId, holder);
         }
 
         if (unstETHRecord.status == UnstETHRecordStatus.NotLocked) {
@@ -389,7 +389,7 @@ library AssetsAccounting {
         if (unstETHRecord.status == UnstETHRecordStatus.Finalized) {
             // if the unstETH was marked finalized earlier, it's claimable amount must stay the same
             if (unstETHRecord.claimableAmount != claimableAmount) {
-                revert InvalidClaimableAmount(unstETHId, claimableAmount, unstETHRecord.claimableAmount);
+                revert InvalidClaimableAmount(unstETHId, claimableAmount);
             }
         } else {
             unstETHRecord.claimableAmount = claimableAmount;
@@ -408,7 +408,7 @@ library AssetsAccounting {
             revert InvalidUnstETHStatus(unstETHId, unstETHRecord.status);
         }
         if (unstETHRecord.lockedBy != holder) {
-            revert InvalidUnstETHHolder(unstETHId, holder, unstETHRecord.lockedBy);
+            revert InvalidUnstETHHolder(unstETHId, holder);
         }
         unstETHRecord.status = UnstETHRecordStatus.Withdrawn;
         amountWithdrawn = unstETHRecord.claimableAmount;

diff --git a/test/unit/libraries/AssetsAccounting.t.sol b/test/unit/libraries/AssetsAccounting.t.sol
@@ -781,9 +781,7 @@ contract AssetsAccountingUnitTests is UnitTest {
         uint256[] memory unstETHIds = new uint256[](1);
         unstETHIds[0] = genRandomUnstEthId(1234);
 
-        vm.expectRevert(
-            abi.encodeWithSelector(AssetsAccounting.InvalidUnstETHHolder.selector, unstETHIds[0], holder, address(0x0))
-        );
+        vm.expectRevert(abi.encodeWithSelector(AssetsAccounting.InvalidUnstETHHolder.selector, unstETHIds[0], holder));
 
         AssetsAccounting.accountUnstETHUnlock(_accountingContext, holder, unstETHIds);
     }
@@ -798,9 +796,7 @@ contract AssetsAccountingUnitTests is UnitTest {
         unstETHIds[0] = genRandomUnstEthId(1234);
         _accountingContext.unstETHRecords[unstETHIds[0]].lockedBy = holder;
 
-        vm.expectRevert(
-            abi.encodeWithSelector(AssetsAccounting.InvalidUnstETHHolder.selector, unstETHIds[0], current, holder)
-        );
+        vm.expectRevert(abi.encodeWithSelector(AssetsAccounting.InvalidUnstETHHolder.selector, unstETHIds[0], current));
 
         AssetsAccounting.accountUnstETHUnlock(_accountingContext, current, unstETHIds);
     }
@@ -1251,12 +1247,7 @@ contract AssetsAccountingUnitTests is UnitTest {
         }
 
         vm.expectRevert(
-            abi.encodeWithSelector(
-                AssetsAccounting.InvalidClaimableAmount.selector,
-                unstETHIds[0],
-                claimableAmounts[0],
-                uint256(claimableAmounts[0]) + 1
-            )
+            abi.encodeWithSelector(AssetsAccounting.InvalidClaimableAmount.selector, unstETHIds[0], claimableAmounts[0])
         );
 
         AssetsAccounting.accountUnstETHClaimed(_accountingContext, unstETHIds, claimableAmountsPrepared);
@@ -1392,9 +1383,7 @@ contract AssetsAccountingUnitTests is UnitTest {
         _accountingContext.unstETHRecords[unstETHIds[0]].lockedBy = holder;
         _accountingContext.unstETHRecords[unstETHIds[0]].claimableAmount = ETHValues.from(123);
 
-        vm.expectRevert(
-            abi.encodeWithSelector(AssetsAccounting.InvalidUnstETHHolder.selector, unstETHIds[0], current, holder)
-        );
+        vm.expectRevert(abi.encodeWithSelector(AssetsAccounting.InvalidUnstETHHolder.selector, unstETHIds[0], current));
 
         AssetsAccounting.accountUnstETHWithdraw(_accountingContext, current, unstETHIds);
     }