fix(dinero#toFormat): Use toRoundedUnit instead of toRounded in toFor…

…mat method to get rounded value
dinerojs · Mar 13, 2018 · 9b2990d · 9b2990d
1 parent 13ba7ba
commit 9b2990d
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 1 deletion.
diff --git a/src/dinero.js b/src/dinero.js
@@ -473,7 +473,9 @@ const Dinero = options => {
     toFormat(format) {
       const formatter = Format(format || globalFormat)
 
-      return this.toUnit().toLocaleString(this.getLocale(), {
+      return this.toRoundedUnit(
+        formatter.getMinimumFractionDigits()
+      ).toLocaleString(this.getLocale(), {
         currencyDisplay: formatter.getCurrencyDisplay(),
         useGrouping: formatter.getUseGrouping(),
         minimumFractionDigits: formatter.getMinimumFractionDigits(),

diff --git a/test/unit/dinero.spec.js b/test/unit/dinero.spec.js
@@ -278,6 +278,11 @@ describe('Dinero', () => {
         Dinero({ amount: 200000, currency: 'EUR' }).toFormat('0.0')
       ).to.equal('2000.0')
     })
+    it('should return the properly formatted amount (one fraction digit, rounded)', () => {
+      expect(
+        Dinero({ amount: 1155, currency: 'EUR' }).toFormat('0.0')
+      ).to.equal('11.6')
+    })
     it('should return the properly formatted amount (use grouping)', () => {
       expect(
         Dinero({ amount: 200000, currency: 'EUR' }).toFormat('0,0')