Enhance NamespaceParser to support '_' prefix for versioning and update tests accordingly

Author: xavierjohn

src/Abstractions/src/Asp.Versioning.Abstractions/NamespaceParser.cs

@@ -157,21 +157,33 @@ protected virtual bool TryParse( Text identifier, out ApiVersion? apiVersion )
             return false;
         }
 
-        // 'v' | 'V' : [<year> : ['_'] : <month> : ['_'] : <day>] : ['_'] : [<major> ['_' : <minor>]] : ['_'] : [<status>]
+        // 'v' | 'V' | '_' : [<year> : ['_'] : <month> : ['_'] : <day>] : ['_'] : [<major> ['_' : <minor>]] : ['_'] : [<status>]
         //
         // - v1
         // - v1_1
         // - v2_0_Beta
         // - v20180401
         // - v2018_04_01_1_1_Beta
+        // - _2018_04_01
         var ch = identifier[0];
 
-        if ( ch != 'v' && ch != 'V' )
+        if ( ch != 'v' && ch != 'V' && ch != '_' )
         {
             apiVersion = default;
             return false;
         }
 
+        if ( ch == '_' )
+        {
+            // '_' prefix requires the next character to be a digit;
+            // otherwise, it's just a regular identifier
+            if ( identifier.Length < 2 || !char.IsDigit( identifier[1] ) )
+            {
+                apiVersion = default;
+                return false;
+            }
+        }
+
 #if NETSTANDARD
         identifier = identifier.Substring( 1 );
 #else

src/Abstractions/test/Asp.Versioning.Abstractions.Tests/NamespaceParserTest.cs

@@ -70,6 +70,16 @@ public void parse_should_return_no_versions()
         { "Contoso.Api.v2018_04_01_1_0_Beta.Controllers", "2018-04-01.1.0-Beta" },
         { "MyRestaurant.Vegetarian.Food.v1_1.Controllers", "1.1" },
         { "VersioningSample.V5.Controllers", "5.0" },
+        { "_1", "1" },
+        { "_1_1", "1.1" },
+        { "_20180401", "2018-04-01" },
+        { "_2018_04_01", "2018-04-01" },
+        { "_2018_04_01_Beta", "2018-04-01-Beta" },
+        { "_2018_04_01_1_0_Beta", "2018-04-01.1.0-Beta" },
+        { "Contoso.Api._2018_04_01.Controllers", "2018-04-01" },
+        { "Contoso.Api._1.Controllers", "1" },
+        { "Contoso.Api._1_1.Controllers", "1.1" },
+        { "Contoso.Api._0_9_Beta.Controllers", "0.9-Beta" },
     };
 
     public static TheoryData<string, string[]> NamespaceWithMultipleVersions => new()

src/Common/test/Common.Mvc.Tests/Conventions/VersionByNamespaceConventionTest.cs

@@ -24,6 +24,16 @@ public partial class VersionByNamespaceConventionTest
         { "Contoso.Api.v2018_04_01_1_0_Beta.Controllers", "2018-04-01.1.0-Beta" },
         { "MyRestaurant.Vegetarian.Food.v1_1.Controllers", "1.1" },
         { "VersioningSample.V5.Controllers", "5.0" },
+        { "_1", "1.0" },
+        { "_1_1", "1.1" },
+        { "_20180401", "2018-04-01" },
+        { "_2018_04_01", "2018-04-01" },
+        { "_2018_04_01_Beta", "2018-04-01-Beta" },
+        { "_2018_04_01_1_0_Beta", "2018-04-01.1.0-Beta" },
+        { "Contoso.Api._2018_04_01.Controllers", "2018-04-01" },
+        { "Contoso.Api._1.Controllers", "1.0" },
+        { "Contoso.Api._1_1.Controllers", "1.1" },
+        { "Contoso.Api._0_9_Beta.Controllers", "0.9-Beta" },
     };
 
     private sealed class TestType : TypeDelegator