From 869825b16c80c2b770603019b6852f87027b9d1c Mon Sep 17 00:00:00 2001 From: Doug Bunting Date: Mon, 12 Mar 2018 15:28:30 -0700 Subject: [PATCH] Quick fix: Add test of #2890 - tests added (ages ago) in aspnet/Mvc@a045324 no longer fail when `__` handling removed --- .../Internal/ExpressionHelperTest.cs | 21 +++++++++++++++++++ 1 file changed, 21 insertions(+) diff --git a/test/Microsoft.AspNetCore.Mvc.ViewFeatures.Test/Internal/ExpressionHelperTest.cs b/test/Microsoft.AspNetCore.Mvc.ViewFeatures.Test/Internal/ExpressionHelperTest.cs index b6df9b1bdd..eee3b4fb66 100644 --- a/test/Microsoft.AspNetCore.Mvc.ViewFeatures.Test/Internal/ExpressionHelperTest.cs +++ b/test/Microsoft.AspNetCore.Mvc.ViewFeatures.Test/Internal/ExpressionHelperTest.cs @@ -435,6 +435,27 @@ namespace Microsoft.AspNetCore.Mvc.ViewFeatures.Internal Assert.NotSame(text1, text2); } + [Fact] + public void GetExpressionText_WithinALoop_ReturnsExpectedText() + { + // Arrange 0 + var collection = new List(); + + for (var i = 0; i < 2; i++) + { + // Arrange i + var expectedText = $"collection[{i}].SelectedCategory.CategoryId"; + + // Act i + var result = ExpressionHelper.GetExpressionText( + (Expression, int>>)(m => collection[i].SelectedCategory.CategoryId), + _expressionTextCache); + + // Assert i + Assert.Equal(expectedText, result); + } + } + private class TestModel { public string Name { get; set; }