Merge pull request #566 from PolarGoose/FIx_an_AsyncActionsFixture_un…

…it_test_and_a_potential_null_reference_exception_in_InvocationInfo_class Fix NullReferenceException in the InvocationInfo class. Fix a FireAsync_TriggerWithMoreThanThreeParameters test failure because of a different system locale.
dotnet-state-machine · Apr 22, 2024 · 5fa762b · 5fa762b
2 parents a7b0f09 + 0c11ccb
commit 5fa762b
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 2 deletions.
diff --git a/src/Stateless/Reflection/InvocationInfo.cs b/src/Stateless/Reflection/InvocationInfo.cs
@@ -63,9 +63,11 @@ public string Description
             {
                 if (_description != null)
                     return _description;
+                if (MethodName == null)
+                    return "<null>";
                 if (MethodName.IndexOfAny(new char[] { '<', '>', '`' }) >= 0)
                     return DefaultFunctionDescription;
-                return MethodName ?? "<null>";
+                return MethodName;
             }
         }
 

diff --git a/test/Stateless.Tests/AsyncActionsFixture.cs b/test/Stateless.Tests/AsyncActionsFixture.cs
@@ -5,6 +5,7 @@
 using System.Collections.Generic;
 
 using Xunit;
+using System.Globalization;
 
 namespace Stateless.Tests
 {
@@ -567,7 +568,8 @@ public async Task FireAsyncTriggerWithParametersArray()
         [Fact]
         public async Task FireAsync_TriggerWithMoreThanThreeParameters()
         {
-            const string expectedParam = "42-Stateless-True-420.69-Y";
+            var decimalSeparator = CultureInfo.CurrentCulture.NumberFormat.NumberDecimalSeparator;
+            string expectedParam = $"42-Stateless-True-420{decimalSeparator}69-Y";
             string actualParam = null;
 
             var sm = new StateMachine<State, Trigger>(State.A);