Add unit tests to cover new LoggingHook

Signed-off-by: Kyle Julian <[email protected]>

Author: kylejuliandev

Directory.Packages.props

@@ -23,6 +23,7 @@
     <PackageVersion Include="coverlet.msbuild" Version="6.0.3" />
     <PackageVersion Include="FluentAssertions" Version="7.0.0" />
     <PackageVersion Include="GitHubActionsTestLogger" Version="2.4.1" />
+    <PackageVersion Include="Microsoft.Extensions.Diagnostics.Testing" Version="9.0.0" />
     <PackageVersion Include="Microsoft.NET.Test.Sdk" Version="17.12.0" />
     <PackageVersion Include="NSubstitute" Version="5.3.0" />
     <PackageVersion Include="SpecFlow" Version="3.9.74" />

src/OpenFeature/Hooks/LoggingHook.cs

@@ -22,6 +22,8 @@ public sealed partial class LoggingHook : Hook
         /// </summary>
         public LoggingHook(ILogger logger, bool includeContext = false)
         {
+            if (logger == null) throw new ArgumentNullException(nameof(logger));
+
             this._logger = logger;
             this._includeContext = includeContext;
         }
@@ -78,19 +80,16 @@ public override ValueTask AfterAsync<T>(HookContext<T> context, FlagEvaluationDe
         }
 
         [LoggerMessage(
-            EventId = 0,
             Level = LogLevel.Debug,
             Message = "Before Flag Evaluation {Content}")]
         partial void HookBeforeStageExecuted(LoggingHookContent content);
 
         [LoggerMessage(
-            EventId = 0,
             Level = LogLevel.Error,
             Message = "Error during Flag Evaluation {Content}")]
         partial void HookErrorStageExecuted(LoggingHookContent content);
 
         [LoggerMessage(
-            EventId = 0,
             Level = LogLevel.Debug,
             Message = "After Flag Evaluation {Content}")]
         partial void HookAfterStageExecuted(LoggingHookContent content);
@@ -166,6 +165,12 @@ public override string ToString()
                 if (value.IsBoolean)
                     return value.AsBoolean.ToString();
 
+                if (value.IsNumber)
+                {
+                    if (value.AsDouble != null) return value.AsDouble.ToString();
+                    if (value.AsInteger != null) return value.AsInteger.ToString();
+                }
+
                 if (value.IsDateTime)
                     return value.AsDateTime?.ToString("O");