Merge pull request #6383 from aspnet/rel/2.0.0-preview2

Modify ControllerBase.RedirectToPageResult to call the correct overload

src/Microsoft.AspNetCore.Mvc.Core/ControllerBase.cs

@@ -983,7 +983,7 @@ namespace Microsoft.AspNetCore.Mvc
         /// <returns>The <see cref="RedirectToPageResult"/>.</returns>
         [NonAction]
         public virtual RedirectToPageResult RedirectToPage(string pageName, string pageHandler, object routeValues)
-            => RedirectToPage(pageName, pageHandler, routeValues);
+            => RedirectToPage(pageName, pageHandler, routeValues, fragment: null);
 
         /// <summary>
         /// Redirects (<see cref="StatusCodes.Status302Found"/>) to the specified <paramref name="pageName"/>

test/Microsoft.AspNetCore.Mvc.Core.Test/ControllerBaseTest.cs

@@ -2478,6 +2478,26 @@ namespace Microsoft.AspNetCore.Mvc.Core.Test
             Assert.True(controller.ModelState.IsValid);
         }
 
+        [Fact]
+        public void RedirectToPage_WithPageName_Handler_AndRouteValues()
+        {
+            // Arrange
+            var controller = new TestableController();
+
+            // Act
+            var result = controller.RedirectToPage("page", "handler", new { test = "value"});
+
+            // Assert
+            Assert.Equal("page", result.PageName);
+            Assert.Equal("handler", result.PageHandler);
+            Assert.Collection(result.RouteValues,
+                item =>
+                {
+                    Assert.Equal("test", item.Key);
+                    Assert.Equal("value", item.Value);
+                });
+        }
+
         private static ControllerBase GetController(IModelBinder binder, IValueProvider valueProvider)
         {
             var metadataProvider = TestModelMetadataProvider.CreateDefaultProvider();