Fix Task of IResult for MapAction (#30355)

Author: Kahbazi

src/Http/Routing/src/Internal/MapActionExpressionTreeBuilder.cs

@@ -427,7 +427,7 @@ static async Task ExecuteAwaited(ValueTask<T> task, HttpContext httpContext)
             return ExecuteAwaited(task, httpContext);
         }
 
-        private static async ValueTask ExecuteTaskResult<T>(Task<T> task, HttpContext httpContext) where T : IResult
+        private static async Task ExecuteTaskResult<T>(Task<T> task, HttpContext httpContext) where T : IResult
         {
             await (await task).ExecuteAsync(httpContext);
         }

src/Http/Routing/test/UnitTests/Internal/MapActionExpressionTreeBuilderTest.cs

@@ -581,24 +581,47 @@ public async Task RequestDelegateWritesComplexReturnValueAsJsonResponseBody(Dele
             Assert.Equal("Write even more tests!", deserializedResponseBody!.Name);
         }
 
-        [Fact]
-        public async Task RequestDelegateUsesCustomIResult()
+        public static IEnumerable<object[]> CustomResults
         {
-            var resultString = "Still not enough tests!";
+            get
+            {
+                var resultString = "Still not enough tests!";
+
+                CustomResult TestAction() => new CustomResult(resultString);
+                Task<CustomResult> TaskTestAction() => Task.FromResult(new CustomResult(resultString));
+                ValueTask<CustomResult> ValueTaskTestAction() => ValueTask.FromResult(new CustomResult(resultString));
 
-            CustomResult TestAction() => new(resultString!);
+                static CustomResult StaticTestAction() => new CustomResult("Still not enough tests!");
+                static Task<CustomResult> StaticTaskTestAction() => Task.FromResult(new CustomResult("Still not enough tests!"));
+                static ValueTask<CustomResult> StaticValueTaskTestAction() => ValueTask.FromResult(new CustomResult("Still not enough tests!"));
 
+                return new List<object[]>
+                {
+                    new object[] { (Func<CustomResult>)TestAction },
+                    new object[] { (Func<Task<CustomResult>>)TaskTestAction},
+                    new object[] { (Func<ValueTask<CustomResult>>)ValueTaskTestAction},
+                    new object[] { (Func<CustomResult>)StaticTestAction},
+                    new object[] { (Func<Task<CustomResult>>)StaticTaskTestAction},
+                    new object[] { (Func<ValueTask<CustomResult>>)StaticValueTaskTestAction},
+                };
+            }
+        }
+
+        [Theory]
+        [MemberData(nameof(CustomResults))]
+        public async Task RequestDelegateUsesCustomIResult(Delegate @delegate)
+        {
             var httpContext = new DefaultHttpContext();
             var responseBodyStream = new MemoryStream();
             httpContext.Response.Body = responseBodyStream;
 
-            var requestDelegate = MapActionExpressionTreeBuilder.BuildRequestDelegate((Func<CustomResult>)TestAction);
+            var requestDelegate = MapActionExpressionTreeBuilder.BuildRequestDelegate(@delegate);
 
             await requestDelegate(httpContext);
 
             var decodedResponseBody = Encoding.UTF8.GetString(responseBodyStream.ToArray());
 
-            Assert.Equal(resultString, decodedResponseBody);
+            Assert.Equal("Still not enough tests!", decodedResponseBody);
         }
 
         private class Todo