huahaofeng
/
aspnetcore
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Fix a test based on the resolution of #1609 

This old test (from WSR) was commented out pending the resolution of
issue #1609.

I've updated the test to track the current semantics.
This commit is contained in:
Ryan Nowak 2014-11-26 11:46:05 -08:00
parent 2eef4dd3cf
commit ff3282827a
1 changed files with 13 additions and 24 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

37
test/Microsoft.AspNet.Mvc.ModelBinding.Test/ValueProviders/ReadableStringCollectionValueProviderTest.cs
View File

@ -156,32 +156,21 @@ namespace Microsoft.AspNet.Mvc.ModelBinding.Test
Assert.Equal(culture, vpResult.Culture); Assert.Equal(culture, vpResult.Culture);
} }
// TODO: https://github.com/aspnet/Mvc/issues/1609 [Theory]
// [InlineData("null_value")]
// This test was present in MVC5 and WebAPI2, but our VP implementation doesn't behave this [InlineData("prefix.null_value")]
// way currently. public async Task GetValue_NullValue(string key)
// {
//[Theory] // Arrange
//[InlineData("null_value")] var culture = new CultureInfo("fr-FR");
//[InlineData("prefix.null_value")] var valueProvider = new ReadableStringCollectionValueProvider<TestValueProviderMetadata>(_backingStore, culture);
//public async Task GetValue_NullValue(string key)
//{
// // Arrange
// var culture = new CultureInfo("fr-FR");
// var valueProvider = new ReadableStringCollectionValueProvider<TestValueProviderMetadata>(_backingStore, culture);
// System.Diagnostics.Debugger.Launch(); // Act
// System.Diagnostics.Debugger.Break(); var result = await valueProvider.GetValueAsync(key);
// // Act // Assert
// var result = await valueProvider.GetValueAsync(key); Assert.Null(result);
}
// // Assert
// Assert.NotNull(result);
// Assert.Null(result.RawValue);
// Assert.Null(result.AttemptedValue);
// Assert.Equal(culture, result.Culture);
//}
[Fact] [Fact]
public async Task GetValueAsync_NullMultipleValue() public async Task GetValueAsync_NullMultipleValue()