From ebe81c2978244fa92ff8ac68668c86a69aac5a69 Mon Sep 17 00:00:00 2001 From: Steve Sanderson Date: Mon, 30 Sep 2019 09:22:10 +0100 Subject: [PATCH] Fix test that was illegally holding ParameterView references --- .../test/RazorComponents/HtmlRendererTest.cs | 9 +++++---- 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/src/Mvc/Mvc.ViewFeatures/test/RazorComponents/HtmlRendererTest.cs b/src/Mvc/Mvc.ViewFeatures/test/RazorComponents/HtmlRendererTest.cs index 02086813ff..7a670696fb 100644 --- a/src/Mvc/Mvc.ViewFeatures/test/RazorComponents/HtmlRendererTest.cs +++ b/src/Mvc/Mvc.ViewFeatures/test/RazorComponents/HtmlRendererTest.cs @@ -270,7 +270,7 @@ namespace Microsoft.AspNetCore.Mvc.RazorComponents // Assert Assert.Equal(expectedHtml, result); } - + [Fact] public void RenderComponentAsync_MarksSelectedOptionsAsSelected() { @@ -668,18 +668,19 @@ namespace Microsoft.AspNetCore.Mvc.RazorComponents public Task SetParametersAsync(ParameterView parameters) { - _renderHandle.Render(CreateRenderFragment(parameters)); + var content = parameters.GetValueOrDefault("Value"); + _renderHandle.Render(CreateRenderFragment(content)); return Task.CompletedTask; } - private RenderFragment CreateRenderFragment(ParameterView parameters) + private RenderFragment CreateRenderFragment(string content) { return RenderFragment; void RenderFragment(RenderTreeBuilder rtb) { rtb.OpenElement(1, "span"); - rtb.AddContent(2, parameters.GetValueOrDefault("Value")); + rtb.AddContent(2, content); rtb.CloseElement(); } }