fix: medusa and signature

liquity · Oct 15, 2024 · d60e671 · d60e671
1 parent 0b72301
commit d60e671
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 6 deletions.
diff --git a/src/utils/EncodingDecodingLib.sol b/src/utils/EncodingDecodingLib.sol
@@ -2,12 +2,12 @@
 pragma solidity ^0.8.24;
 
 library EncodingDecodingLib {
-    function encodeLQTYAllocation(uint88 _lqty, uint32 _averageTimestamp) public pure returns (uint224) {
+    function encodeLQTYAllocation(uint88 _lqty, uint32 _averageTimestamp) internal pure returns (uint224) {
         uint224 _value = (uint224(_lqty) << 32) | _averageTimestamp;
         return _value;
     }
 
-    function decodeLQTYAllocation(uint224 _value) public pure returns (uint88, uint32) {
+    function decodeLQTYAllocation(uint224 _value) internal pure returns (uint88, uint32) {
         return (uint88(_value >> 32), uint32(_value));
     }
 }
diff --git a/test/recon/properties/BribeInitiativeProperties.sol b/test/recon/properties/BribeInitiativeProperties.sol
@@ -26,8 +26,8 @@ abstract contract BribeInitiativeProperties is BeforeAfter {
                 uint88 lusdPercentageOfBribe88 = uint88(lusdPercentageOfBribe >> 40);
 
                 // calculate user allocation percentage of total for this epoch
-                uint88 lqtyAllocatedByUserAtEpoch = IBribeInitiative(initiative).lqtyAllocatedByUserAtEpoch(user, currentEpoch);
-                uint88 totalLQTYAllocatedAtEpoch = IBribeInitiative(initiative).totalLQTYAllocatedByEpoch(currentEpoch);
+                (uint88 lqtyAllocatedByUserAtEpoch, ) = IBribeInitiative(initiative).lqtyAllocatedByUserAtEpoch(user, currentEpoch);
+                (uint88 totalLQTYAllocatedAtEpoch, ) = IBribeInitiative(initiative).totalLQTYAllocatedByEpoch(currentEpoch);
                 uint88 allocationPercentageOfTotal = (lqtyAllocatedByUserAtEpoch / totalLQTYAllocatedAtEpoch) * 10_000;
 
                 // check that allocation percentage and received bribe percentage match
@@ -45,7 +45,7 @@ abstract contract BribeInitiativeProperties is BeforeAfter {
         uint16 currentEpoch = governance.epoch();
         for(uint8 i; i < deployedInitiatives.length; i++) {
             IBribeInitiative initiative = IBribeInitiative(deployedInitiatives[i]);
-            uint88 lqtyAllocatedByUserAtEpoch = initiative.lqtyAllocatedByUserAtEpoch(user, currentEpoch);
+            (uint88 lqtyAllocatedByUserAtEpoch, ) = initiative.lqtyAllocatedByUserAtEpoch(user, currentEpoch);
             eq(ghostLqtyAllocationByUserAtEpoch[user], lqtyAllocatedByUserAtEpoch, "BI-03: Accounting for user allocation amount is always correct");
         }
     }
@@ -54,7 +54,7 @@ abstract contract BribeInitiativeProperties is BeforeAfter {
         uint16 currentEpoch = governance.epoch();
         for(uint8 i; i < deployedInitiatives.length; i++) {
             IBribeInitiative initiative = IBribeInitiative(deployedInitiatives[i]);
-            uint88 totalLQTYAllocatedAtEpoch = initiative.totalLQTYAllocatedByEpoch(currentEpoch);
+            (uint88 totalLQTYAllocatedAtEpoch, ) = initiative.totalLQTYAllocatedByEpoch(currentEpoch);
             eq(ghostTotalAllocationAtEpoch[currentEpoch], totalLQTYAllocatedAtEpoch, "BI-04: Accounting for total allocation amount is always correct");
         }
     }