Divide by 10 instead of subtracting one from exponent. (#545)

SetProtocol · Dec 22, 2019 · 25349c3 · 25349c3
1 parent 00971a7
commit 25349c3
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 4 deletions.
diff --git a/contracts/core/liquidators/impl/Auction.sol b/contracts/core/liquidators/impl/Auction.sol
@@ -90,7 +90,7 @@ contract Auction {
         internal
     {
         _auction.pricePrecision = calculatePricePrecision(_currentSet, _nextSet);
-
+        
         uint256 minimumBid = calculateMinimumBid(_currentSet, _nextSet, _auction.pricePrecision);
 
         // remainingCurrentSets must be greater than minimumBid or no bidding would be allowed
@@ -216,7 +216,7 @@ contract Auction {
 
             // Apply order of magnitude to pricePrecision, only want increase if value is >10x so subtract
             // one order of magnitude
-            return MINIMUM_PRICE_PRECISION.mul(10 ** orderOfMagnitude.sub(1));
+            return MINIMUM_PRICE_PRECISION.mul(10 ** orderOfMagnitude).div(10);
         }
 
         return MINIMUM_PRICE_PRECISION;

diff --git a/contracts/lib/CommonMath.sol b/contracts/lib/CommonMath.sol
@@ -156,7 +156,7 @@ library CommonMath {
         // Make sure passed value is greater than 0
         require (
             _value > 0,
-            "AllocatorMathLibrary.ceilLog10: Value must be greater than zero."
+            "CommonMath.ceilLog10: Value must be greater than zero."
         );
 
         // Since log10(1) = 0, if _value = 1 return 0

diff --git a/test/contracts/viewer/lib/rebalancingSetTokenViewer.spec.ts b/test/contracts/viewer/lib/rebalancingSetTokenViewer.spec.ts
@@ -374,7 +374,7 @@ contract('ProtocolViewer', accounts => {
     });
   });
 
-  describe.only('#batchFetchRebalanceStateAsync', async () => {
+  describe('#batchFetchRebalanceStateAsync', async () => {
     let subjectRebalancingSetAddresses: Address[];
 
     let rebalancingSetToken: RebalancingSetTokenContract;