Fix MSBuild handling of cultures plus add unit tests

Fixes #5847

1 files changed, 3 insertions, 5 deletions

diff --git a/src/WixToolset.BuildTasks/WixAssignCulture.cs b/src/WixToolset.BuildTasks/WixAssignCulture.cs
index 7a03dc47..a8baa62f 100644
--- a/src/WixToolset.BuildTasks/WixAssignCulture.cs
+++ b/src/WixToolset.BuildTasks/WixAssignCulture.cs
@@ -174,8 +174,6 @@ namespace WixToolset.BuildTasks
 
         private class CultureGroup
         {
-            private List<string> cultures = new List<string>();
-
             /// <summary>
             /// TargetPath already has a '\', do not double it!
             /// </summary>
@@ -193,11 +191,11 @@ namespace WixToolset.BuildTasks
                 Debug.Assert(!String.IsNullOrEmpty(cultureGroupString));
                 foreach (string cultureString in cultureGroupString.Split(','))
                 {
-                    this.cultures.Add(cultureString);
+                    this.Cultures.Add(cultureString);
                 }
             }
 
-            public List<string> Cultures { get { return cultures; } }
+            public List<string> Cultures { get; } = new List<string>();
 
             public string OutputFolder
             {
@@ -218,7 +216,7 @@ namespace WixToolset.BuildTasks
             {
                 if (this.Cultures.Count > 0)
                 {
-                    return String.Join(",", this.Cultures.ToArray());
+                    return String.Join(";", this.Cultures);
                 }
 
                 // We use a keyword for a null culture because MSBuild cannnot handle "" items