fix: Use CurrencyDecimalDigits instead of NumberDecimalDigits when ro…

…unding money
VirtoCommerce · Nov 15, 2024 · b0dd10c · b0dd10c
1 parent 52de3ad
commit b0dd10c
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/src/VirtoCommerce.CoreModule.Core/Currency/DefaultMoneyRoundingPolicy.cs b/src/VirtoCommerce.CoreModule.Core/Currency/DefaultMoneyRoundingPolicy.cs
@@ -12,7 +12,7 @@ public decimal RoundMoney(decimal amount, Currency currency)
         {
             var roundingType = EnumUtility.SafeParse(currency.RoundingType, RoundingType.Rounding001);
             var midpointRounding = EnumUtility.SafeParse(currency.MidpointRounding, MidpointRounding.AwayFromZero);
-            return Round(amount, currency.NumberFormat.NumberDecimalDigits, roundingType, midpointRounding);
+            return Round(amount, currency.NumberFormat.CurrencyDecimalDigits, roundingType, midpointRounding);
         }
 
         /// <summary>

diff --git a/tests/VirtoCommerce.CoreModule.Tests/DefaultMoneyRoundingPolicyTests.cs b/tests/VirtoCommerce.CoreModule.Tests/DefaultMoneyRoundingPolicyTests.cs
@@ -1,5 +1,6 @@
 using System;
 using System.Collections.Generic;
+using VirtoCommerce.CoreModule.Core.Common;
 using VirtoCommerce.CoreModule.Core.Currency;
 using VirtoCommerce.CoreModule.Core.Enums;
 using Xunit;
@@ -14,7 +15,7 @@ public void CanRound(decimal amount, decimal expected, RoundingType roundingType
         {
             // Arrange
             var roundPolicy = new DefaultMoneyRoundingPolicy();
-            var currency = new Currency();
+            var currency = new Currency(new Language("en-US"), code: null);
             currency.RoundingType = roundingType.ToString();
             currency.MidpointRounding = midpointRounding.ToString();