diff --git a/src/Microsoft.AspNet.Mvc.Core/AntiForgery/AntiForgeryTokenStore.cs b/src/Microsoft.AspNet.Mvc.Core/AntiForgery/AntiForgeryTokenStore.cs index d9576fe4dc..4395181cd4 100644 --- a/src/Microsoft.AspNet.Mvc.Core/AntiForgery/AntiForgeryTokenStore.cs +++ b/src/Microsoft.AspNet.Mvc.Core/AntiForgery/AntiForgeryTokenStore.cs @@ -42,7 +42,7 @@ namespace Microsoft.AspNet.Mvc public async Task GetFormTokenAsync(HttpContext httpContext) { - var form = await httpContext.Request.GetFormAsync(); + var form = await httpContext.Request.ReadFormAsync(); var value = form[_config.FormFieldName]; if (string.IsNullOrEmpty(value)) { diff --git a/src/Microsoft.AspNet.Mvc.ModelBinding/ValueProviders/FormValueProviderFactory.cs b/src/Microsoft.AspNet.Mvc.ModelBinding/ValueProviders/FormValueProviderFactory.cs index 7c3a4b282c..a46d7115b3 100644 --- a/src/Microsoft.AspNet.Mvc.ModelBinding/ValueProviders/FormValueProviderFactory.cs +++ b/src/Microsoft.AspNet.Mvc.ModelBinding/ValueProviders/FormValueProviderFactory.cs @@ -20,7 +20,7 @@ namespace Microsoft.AspNet.Mvc.ModelBinding { var culture = GetCultureInfo(request); return new ReadableStringCollectionValueProvider( - () => request.GetFormAsync(), + async () => await request.ReadFormAsync(), culture); } diff --git a/test/Microsoft.AspNet.Mvc.Core.Test/AntiXsrf/AntiForgeryTokenStoreTest.cs b/test/Microsoft.AspNet.Mvc.Core.Test/AntiXsrf/AntiForgeryTokenStoreTest.cs index 6f66ff63a6..650efd2c82 100644 --- a/test/Microsoft.AspNet.Mvc.Core.Test/AntiXsrf/AntiForgeryTokenStoreTest.cs +++ b/test/Microsoft.AspNet.Mvc.Core.Test/AntiXsrf/AntiForgeryTokenStoreTest.cs @@ -164,13 +164,12 @@ namespace Microsoft.AspNet.Mvc.Core.Test // Arrange var mockHttpContext = new Mock(); var requestContext = new Mock(); - IReadableStringCollection formsCollection = - new MockCookieCollection(new Dictionary() { { "form-field-name", string.Empty } }); - requestContext.Setup(o => o.GetFormAsync(CancellationToken.None)) - .Returns(Task.FromResult(formsCollection)); + var formCollection = new Mock(); + formCollection.Setup(f => f["form-field-name"]).Returns(string.Empty); + requestContext.Setup(o => o.ReadFormAsync(CancellationToken.None)) + .Returns(Task.FromResult(formCollection.Object)); mockHttpContext.Setup(o => o.Request) .Returns(requestContext.Object); - var config = new AntiForgeryOptions() { FormFieldName = "form-field-name" @@ -191,12 +190,12 @@ namespace Microsoft.AspNet.Mvc.Core.Test public async Task GetFormToken_FormFieldIsInvalid_PropagatesException() { // Arrange - IReadableStringCollection formsCollection = - new MockCookieCollection(new Dictionary() { { "form-field-name", "invalid-value" } }); + var formCollection = new Mock(); + formCollection.Setup(f => f["form-field-name"]).Returns("invalid-value"); var requestContext = new Mock(); - requestContext.Setup(o => o.GetFormAsync(CancellationToken.None)) - .Returns(Task.FromResult(formsCollection)); + requestContext.Setup(o => o.ReadFormAsync(CancellationToken.None)) + .Returns(Task.FromResult(formCollection.Object)); var mockHttpContext = new Mock(); mockHttpContext.Setup(o => o.Request) @@ -233,10 +232,10 @@ namespace Microsoft.AspNet.Mvc.Core.Test // Arrange var mockHttpContext = new Mock(); var requestContext = new Mock(); - IReadableStringCollection formsCollection = - new MockCookieCollection(new Dictionary() { { "form-field-name", "valid-value" } }); - requestContext.Setup(o => o.GetFormAsync(CancellationToken.None)) - .Returns(Task.FromResult(formsCollection)); + var formCollection = new Mock(); + formCollection.Setup(f => f["form-field-name"]).Returns("valid-value"); + requestContext.Setup(o => o.ReadFormAsync(CancellationToken.None)) + .Returns(Task.FromResult(formCollection.Object)); mockHttpContext.Setup(o => o.Request) .Returns(requestContext.Object); diff --git a/test/Microsoft.AspNet.Mvc.FunctionalTests/Logging/LoggingExtensions.cs b/test/Microsoft.AspNet.Mvc.FunctionalTests/Logging/LoggingExtensions.cs index e647a38c60..aeed866057 100644 --- a/test/Microsoft.AspNet.Mvc.FunctionalTests/Logging/LoggingExtensions.cs +++ b/test/Microsoft.AspNet.Mvc.FunctionalTests/Logging/LoggingExtensions.cs @@ -155,7 +155,7 @@ namespace Microsoft.AspNet.Mvc.FunctionalTests { var queryString = QueryHelpers.ParseQuery(query); - return queryString[key]; + return queryString[key].FirstOrDefault(); } } } \ No newline at end of file diff --git a/test/Microsoft.AspNet.Mvc.ModelBinding.Test/ValueProviders/FormValueProviderFactoryTests.cs b/test/Microsoft.AspNet.Mvc.ModelBinding.Test/ValueProviders/FormValueProviderFactoryTests.cs index 4796dc7005..a28d096ced 100644 --- a/test/Microsoft.AspNet.Mvc.ModelBinding.Test/ValueProviders/FormValueProviderFactoryTests.cs +++ b/test/Microsoft.AspNet.Mvc.ModelBinding.Test/ValueProviders/FormValueProviderFactoryTests.cs @@ -48,9 +48,9 @@ namespace Microsoft.AspNet.Mvc.ModelBinding.Test private static ValueProviderFactoryContext CreateContext(string contentType) { - var collection = Mock.Of(); + var collection = Mock.Of(); var request = new Mock(); - request.Setup(f => f.GetFormAsync(CancellationToken.None)).Returns(Task.FromResult(collection)); + request.Setup(f => f.ReadFormAsync(CancellationToken.None)).Returns(Task.FromResult(collection)); request.SetupGet(r => r.ContentType).Returns(contentType); var context = new Mock(); diff --git a/test/Microsoft.AspNet.Mvc.ModelBinding.Test/ValueProviders/ReadableStringCollectionValueProviderTest.cs b/test/Microsoft.AspNet.Mvc.ModelBinding.Test/ValueProviders/ReadableStringCollectionValueProviderTest.cs index d878f4e1a3..ff6440dc66 100644 --- a/test/Microsoft.AspNet.Mvc.ModelBinding.Test/ValueProviders/ReadableStringCollectionValueProviderTest.cs +++ b/test/Microsoft.AspNet.Mvc.ModelBinding.Test/ValueProviders/ReadableStringCollectionValueProviderTest.cs @@ -6,7 +6,7 @@ using System.Globalization; using System.Linq; using System.Threading.Tasks; using Microsoft.AspNet.Http; -using Microsoft.AspNet.WebUtilities.Collections; +using Microsoft.AspNet.PipelineCore.Collections; using Xunit; namespace Microsoft.AspNet.Mvc.ModelBinding.Test