Fix Ledger exercise NewLine & CultureInfo issues (#1275)

* Fix Ledger exercise NewLine & CultureInfo issues

- Replace System.Environment.NewLine with \n
- Use new CultureInfo(locale, false) instead
- Add currencyNegativePattern function for the tests

* Replace \n instead to not re-run tests

[no important files changed]

exercises/practice/ledger/.meta/Example.fs

@@ -13,9 +13,15 @@ let currencySymbol =
     | "EUR" -> "€"
     | _ -> failwith "Invalid currency"
 
+let currencyNegativePattern =
+    function
+    | "en-US" -> 0
+    | "nl-NL" -> 12
+    | _ -> failwith "Invalid locale"
+
 let cultureInfo (locale: string) =
     match locale with
-    | "en-US" | "nl-NL" -> new CultureInfo(locale) 
+    | "en-US" | "nl-NL" -> new CultureInfo(locale, false) 
     | _ -> failwith "Invalid locale"
 
 let dateFormat =
@@ -27,6 +33,7 @@ let dateFormat =
 let getCulture (currency: string) (locale: string) =
     let mutable culture = cultureInfo locale
     culture.NumberFormat.CurrencySymbol <- currencySymbol currency
+    culture.NumberFormat.CurrencyNegativePattern <- currencyNegativePattern locale
     culture.DateTimeFormat.ShortDatePattern <- dateFormat locale
     culture
 
@@ -63,4 +70,4 @@ let formatLedger currency locale entries =
     let culture = getCulture currency locale
     let header = printHeader culture
     let lines = List.map (printEntry culture) (orderEntries entries)
-    header :: lines |> String.concat "\n"
+    header :: lines |> String.concat System.Environment.NewLine

exercises/practice/ledger/Ledger.fs

@@ -16,7 +16,7 @@ let formatLedger currency locale entries =
 
     for x in List.sortBy (fun x -> x.dat, x.des, x.chg) entries do
 
-        res <- res + "\n"
+        res <- res + System.Environment.NewLine
 
         if locale = "nl-NL" then 
             res <- res + x.dat.ToString("dd-MM-yyyy")
@@ -39,23 +39,23 @@ let formatLedger currency locale entries =
         if c < 0.0 then 
             if locale = "nl-NL" then
                 if currency = "USD" then
-                    res <- res + ("$ " + c.ToString("#,#0.00", new CultureInfo("nl-NL"))).PadLeft(13) 
+                    res <- res + ("$ " + c.ToString("#,#0.00", new CultureInfo("nl-NL", false))).PadLeft(13) 
                 if currency = "EUR" then
-                    res <- res + ("€ " + c.ToString("#,#0.00", new CultureInfo("nl-NL"))).PadLeft(13) 
+                    res <- res + ("€ " + c.ToString("#,#0.00", new CultureInfo("nl-NL", false))).PadLeft(13) 
             if locale = "en-US" then
                 if currency = "USD" then
-                    res <- res + ("($" + c.ToString("#,#0.00", new CultureInfo("en-US")).Substring(1) + ")").PadLeft(13) 
+                    res <- res + ("($" + c.ToString("#,#0.00", new CultureInfo("en-US", false)).Substring(1) + ")").PadLeft(13) 
                 if currency = "EUR" then
-                    res <- res + ("(€" + c.ToString("#,#0.00", new CultureInfo("en-US")).Substring(1) + ")").PadLeft(13) 
+                    res <- res + ("(€" + c.ToString("#,#0.00", new CultureInfo("en-US", false)).Substring(1) + ")").PadLeft(13) 
         else 
             if locale = "nl-NL" then
                 if currency = "USD" then
-                    res <- res + ("$ " + c.ToString("#,#0.00", new CultureInfo("nl-NL")) + " ").PadLeft(13) 
+                    res <- res + ("$ " + c.ToString("#,#0.00", new CultureInfo("nl-NL", false)) + " ").PadLeft(13) 
                 if currency = "EUR" then
-                    res <- res + ("€ " + c.ToString("#,#0.00", new CultureInfo("nl-NL")) + " ").PadLeft(13) 
+                    res <- res + ("€ " + c.ToString("#,#0.00", new CultureInfo("nl-NL", false)) + " ").PadLeft(13) 
             if locale = "en-US" then
                 if currency = "USD" then
-                    res <- res + ("$" + c.ToString("#,#0.00", new CultureInfo("en-US")) + " ").PadLeft(13) 
+                    res <- res + ("$" + c.ToString("#,#0.00", new CultureInfo("en-US", false)) + " ").PadLeft(13) 
                 if currency = "EUR" then
-                    res <- res + ("€" + c.ToString("#,#0.00", new CultureInfo("en-US")) + " ").PadLeft(13) 
+                    res <- res + ("€" + c.ToString("#,#0.00", new CultureInfo("en-US", false)) + " ").PadLeft(13) 
     res