Fix NullReferenceException in the InvocationInfo class. Fix a FireAsync_TriggerWithMoreThanThreeParameters test failure because of a different system locale.

dotnet-state-machine / stateless

A simple library for creating state machines in C# code

Other

5.53k stars 765 forks source link

Closed PolarGoose closed 5 months ago

PolarGoose commented 8 months ago

The InvocationInfo class is updated to handle cases where MethodName is null. Previously, accessing the Description property could throw a NullReferenceException if MethodName was null.
FireAsync_TriggerWithMoreThanThreeParameters unit test has been modified to be culturally invariant. Before this change, the test could fail on systems with locales that use a decimal separator other than a period ., such as a comma ,.

mclift commented 5 months ago

Thanks for your contribution, @PolarGoose!