Make MoneyFormatter().formatCurrency unit test pass

Author: mindgraffiti

WooCommerce/Classes/Tools/Currency/MoneyFormatter.swift

@@ -122,10 +122,22 @@ extension MoneyFormatter {
     ///     - localizedAmount: a formatted string returned from `localizeAmount()`
     ///     - currencyPosition: the currency position, either right, left, right_space, or left_space.
     ///     - currencyCode: the three-letter country code used for a currency, e.g. CAD.
-
-    // -TODO: currency position should be an enum somewhere. Maybe in the Money struct?
-    func formatCurrency(using localizedAmount: String, positionedAt currencyPosition: String, with currencyCode: Currency.Code) -> String? {
-        return ""
+    func formatCurrency(using amount: String,
+                        at position: Currency.Position,
+                        with code: Currency.Code) -> String? {
+        let currency = Currency(amount: amount, code: code, position: position)
+        let symbol = currency.symbol
+
+        switch position {
+        case .left:
+            return symbol + amount
+        case .right:
+            return amount + symbol
+        case .leftSpace:
+            return symbol + "\u{00a0}" + amount
+        case .rightSpace:
+            return amount + "\u{00a0}" + symbol
+        }
     }
 
 

WooCommerce/WooCommerceTests/Tools/MoneyFormatterTests.swift

@@ -476,7 +476,7 @@ extension MoneyFormatter {
         let decimalSeparator = ","
         let thousandSeparator = "."
         let decimalPosition = 3
-        let currencyPosition = "right_space"
+        let currencyPosition = Currency.Position.rightSpace
         let currencyCode = Currency.Code.JOD
         let stringAmount = "-7867818684.64"
         let expectedResult = "-7.867.818.684,640 د.ا"
@@ -499,7 +499,7 @@ extension MoneyFormatter {
         }
 
         let formattedAmount = MoneyFormatter().formatCurrency(using: localizedAmount,
-                                                              positionedAt: currencyPosition,
+                                                              at: currencyPosition,
                                                               with: currencyCode)
 
         guard let actualResult = formattedAmount else {