Add tests for all current BCL cultures, and expand parser to support three-letter locales

Author: baronfel

src/NuGet.Core/NuGet.Packaging/ContentModel/ManagedCodeConventions.cs

@@ -160,7 +160,7 @@ private static object CodeLanguage_Parser(ReadOnlyMemory<char> name, PatternTabl
         /// If matchOnly is true, then an empty string may be returned as a performance optimization.
         /// If matchOnly is false, the parsed result will be returned.
         /// </summary>
-        private static object Locale_Parser(ReadOnlyMemory<char> name, PatternTable table, bool matchOnly)
+        internal static object Locale_Parser(ReadOnlyMemory<char> name, PatternTable table, bool matchOnly)
         {
             if (table != null)
             {
@@ -173,7 +173,8 @@ private static object Locale_Parser(ReadOnlyMemory<char> name, PatternTable tabl
 
             // We use a heuristic here for common locale codes. Locale codes are often
             // * two characters for the language: en, es, fr, de
-            if (name.Length == 2)
+            // * three characters for the language: agq
+            if (name.Length == 2 || name.Length == 3)
             {
                 if (matchOnly)
                 {
@@ -193,6 +194,10 @@ private static object Locale_Parser(ReadOnlyMemory<char> name, PatternTable tabl
             }
             else if (name.Length >= 5 && name.Span[3] == '-') // e.g agq-CM
             {
+                if (matchOnly)
+                {
+                    return string.Empty;
+                }
                 return name;
             }
 

test/NuGet.Core.Tests/NuGet.Packaging.Test/ContentModelTests/ContentModelResourceTests.cs

@@ -0,0 +1,28 @@
+// Copyright (c) .NET Foundation. All rights reserved.
+// Licensed under the Apache License, Version 2.0. See License.txt in the project root for license information.
+
+using System;
+using System.Collections.Generic;
+using System.Globalization;
+using System.Linq;
+using Xunit;
+
+namespace NuGet.Client.Test
+{
+    public class ContentModelResourceTests
+    {
+        [MemberData(nameof(AllCultures))]
+        [Theory]
+        public void CanParseEverySystemKnownCultureResource(CultureInfo culture)
+        {
+            var result = ManagedCodeConventions.Locale_Parser(culture.Name.AsMemory(), null, false);
+            Assert.Equal(culture.Name, result as string);
+        }
+
+        public static IEnumerable<object[]> AllCultures()
+        {
+            return CultureInfo.GetCultures(CultureTypes.AllCultures).Where(c => !string.IsNullOrEmpty(c.Name)).Select(culture => new[] { culture });
+        }
+    }
+}
+