bankers rounding

LedgerHQ · Sep 5, 2024 · 02e4bdd · 02e4bdd
1 parent aa01899
commit 02e4bdd
Showing 1 changed file with 3 additions and 1 deletion.
diff --git a/apps/ledger-live-mobile/e2e/specs/delegate/cosmos.spec.ts b/apps/ledger-live-mobile/e2e/specs/delegate/cosmos.spec.ts
@@ -38,10 +38,12 @@ describe("Cosmos delegate flow", () => {
     const unit = getAccountUnit(testAccount);
 
     const usableAmount = testAccount.spendableBalance.minus(COSMOS_MIN_SAFE).minus(COSMOS_MIN_FEES);
+    // Banker's rounding to avoid floating point errors
+    // NOTE: we could allow for some precision error here to avoid rounding issues 
     const delegatedAmount = usableAmount
       .multipliedBy(delegatedPercent)
       .div(100)
-      .integerValue(BigNumber.ROUND_CEIL);
+      .integerValue(BigNumber.ROUND_HALF_EVEN);
     const remainingAmount = usableAmount.minus(delegatedAmount);
 
     await app.stake.selectCurrency(testedCurrency);