Start adding EndpointMethodInfoApiDescriptionProviderTests

Author: halter73

src/Mvc/Mvc.ApiExplorer/src/EndpointMethodInfoApiDescriptionProvider.cs

@@ -52,7 +52,7 @@ public void OnProvidersExecuted(ApiDescriptionProviderContext context)
         {
         }
 
-        private static ApiDescription CreateApiDescription(RoutePattern pattern, string httpMethod, MethodInfo methodInfo)
+        internal static ApiDescription CreateApiDescription(RoutePattern pattern, string httpMethod, MethodInfo methodInfo)
         {
             var apiDescription = new ApiDescription
             {

src/Mvc/Mvc.ApiExplorer/test/EndpointMethodInfoApiDescriptionProviderTest.cs

@@ -0,0 +1,62 @@
+// 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.Reflection;
+using System.Threading.Tasks;
+using Microsoft.AspNetCore.Http;
+using Microsoft.AspNetCore.Mvc.Abstractions;
+using Microsoft.AspNetCore.Routing;
+using Microsoft.AspNetCore.Routing.Patterns;
+using Xunit;
+
+namespace Microsoft.AspNetCore.Mvc.ApiExplorer
+{
+    public class EndpointMethodInfoApiDescriptionProviderTest
+    {
+        [Fact]
+        public void ApiDescription_MultipleCreatedForMultipleHttpMethods()
+        {
+            Action action = () => { };
+            var httpMethods = new string[] { "FOO", "BAR" };
+
+            var apiDescriptions = GetApiDescriptions(action.Method, httpMethods);
+
+            Assert.Equal(httpMethods.Length, apiDescriptions.Count);
+        }
+
+        [Fact]
+        public void ApiDescription_NotCreatedIfNoHttpMethods()
+        {
+            Action action = () => { };
+            var emptyHttpMethods = new string[0];
+
+            var apiDescriptions = GetApiDescriptions(action.Method, emptyHttpMethods);
+
+            Assert.Empty(apiDescriptions);
+        }
+
+        private IList<ApiDescription> GetApiDescriptions(
+            MethodInfo methodInfo,
+            IEnumerable<string> httpMethods = null,
+            string pattern = null)
+        {
+            var context = new ApiDescriptionProviderContext(Array.Empty<ActionDescriptor>());
+
+            var httpMethodMetadata = new HttpMethodMetadata(httpMethods ?? new[] { "GET" });
+            var endpointMetadata = new EndpointMetadataCollection(methodInfo, httpMethodMetadata);
+            var routePattern = RoutePatternFactory.Pattern(pattern ?? "/");
+
+            var endpoint = new RouteEndpoint(httpContext => Task.CompletedTask, routePattern, 0, endpointMetadata, null);
+            var endpointDataSource = new DefaultEndpointDataSource(endpoint);
+
+            var provider = new EndpointMethodInfoApiDescriptionProvider(endpointDataSource);
+
+            provider.OnProvidersExecuting(context);
+            provider.OnProvidersExecuted(context);
+
+            return context.Results;
+        }
+    }
+}

src/Shared/RequestDelegateFactoryUtilities.cs

@@ -12,7 +12,7 @@ internal static class RequestDelegateFactoryUtilities
     {
         private static readonly MethodInfo EnumTryParseMethod = GetEnumTryParseMethod();
 
-        // Since this is shared source the cache won't be shared between RequestDelegateFactory and the ApiDescriptionProvider sadly :(
+        // Since this is shared source, the cache won't be shared between RequestDelegateFactory and the ApiDescriptionProvider sadly :(
         private static readonly ConcurrentDictionary<Type, MethodInfo?> TryParseMethodCache = new();
 
         public static bool HasTryParseMethod(ParameterInfo parameter)