SetParametersAndRender will breaks with .NET 8

[a2er]

Describe the bug
The extension function SetParametersAndRender throws a NullPointerException since the ComponentState.SetDirectParameters methods vilisbility is changed to internal in .NET 8.0 Preview 4.

The exception is raised here:

bUnit/src/bunit.core/Rendering/TestRenderer.cs

Lines 211 to 213 in e654d6d

	var componentState = GetRequiredComponentStateMethod.Invoke(this, new object[] { renderedComponent.ComponentId })!;
	var setDirectParametersMethod = componentState.GetType().GetMethod("SetDirectParameters", BindingFlags.Public | BindingFlags.Instance)!;
	setDirectParametersMethod.Invoke(componentState, new object[] { parameters });

Example:
Testing this component:

<div>
    @Test
</div>

@code {
    [Parameter]
    public string? Test { get; set; }
}

With this test:

        // Arrange
        var test = "Hello";
        var nextTest = "World";

        // Act
        var cut = RenderComponent<ExampleComponent>(p => p.Add(x => x.Test, test));
        cut.SetParametersAndRender(p => p.Add(x => x.Test, nextTest));

        // Assert
        cut.MarkupMatches($"<div>{nextTest}</div>");

Results in this output:

Message: 
System.NullReferenceException : Object reference not set to an instance of an object.

  Stack Trace: 
<>c__DisplayClass32_0.<SetDirectParameters>b__0() line 213
<>c.<InvokeAsync>b__8_0(Object state)
--- End of stack trace from previous location ---
TestRenderer.AssertNoUnhandledExceptions() line 582
TestRenderer.SetDirectParameters(IRenderedFragmentBase renderedComponent, ParameterView parameters) line 232
RenderedComponentRenderExtensions.SetParametersAndRender[TComponent](IRenderedComponentBase`1 renderedComponent, ParameterView parameters) line 33
RenderedComponentRenderExtensions.SetParametersAndRender[TComponent](IRenderedComponentBase`1 renderedComponent, Action`1 parameterBuilder) line 68
Tests.UpdatesCorrectly() line 39
RuntimeMethodHandle.InvokeMethod(Object target, Void** arguments, Signature sig, Boolean isConstructor)
MethodInvoker.Invoke(Object obj, IntPtr* args, BindingFlags invokeAttr)

Version info:

• bUnit version: 1.22.19
• .NET Runtime and Blazor version: 8.0.0-preview.6.23329.11

Additional context:

Related changes in dotnet/aspnetcore:

• Commit: #ff2241b
• Pull Request: 47555

[linkdotnet]

Hey @a2er,

Yes that is definitely a regression we did not catch as we only run the test up til net7.0.
@egil as v2 is not ready, we should enable v1 for net8.0. At least to a degree where users can still use v1 without any issue on the newest .net version.

[linkdotnet]

@egil: I have a PR ready when I get back from the holidays (tomorrow evening).
I have a new laptop and my ssh settings and stuff are not ported - therefore I want to set this up tomorrow for signed commits and stuff

[egil]

FYI: In net 8 some of the things we need reflection for are now available without.

[a2er]

Thanks for the quick respone!

FYI: I was running the unit tests with .net 8. There are other issues:

(Fails related to this issue are not shown.)

Let me know if you want me to do further investigations on those.

[linkdotnet]

Thanks for the heads up. Yes there are other things for net8.0. Mainly the renderer has a new method that the bUnit one has to implement. And there is a new test double we have to provide.

[linkdotnet]

There should be a prerelease on NuGet soon.
Would be nice to get some feedback on this one

https://www.nuget.org/packages/bunit/1.23.1-preview

[a2er]

SetParametersAndRender is working as expected.
I will continue playing around with .NET 8 Blazor (but WASM only) you may hear from me.

Thanks for the quick fix guys!