bug: Should map metadata when converting from ResolutionDetails to FlagEvaluationDetails

Signed-off-by: Benjamin Evenson <[email protected]>

Author: benjiro

src/OpenFeature/Extension/ResolutionDetailsExtensions.cs

@@ -7,7 +7,7 @@ internal static class ResolutionDetailsExtensions
         public static FlagEvaluationDetails<T> ToFlagEvaluationDetails<T>(this ResolutionDetails<T> details)
         {
             return new FlagEvaluationDetails<T>(details.FlagKey, details.Value, details.ErrorType, details.Reason,
-                details.Variant, details.ErrorMessage);
+                details.Variant, details.ErrorMessage, details.FlagMetadata);
         }
     }
 }

test/OpenFeature.Tests/OpenFeatureClientTests.cs

@@ -11,6 +11,7 @@
 using NSubstitute.ExceptionExtensions;
 using OpenFeature.Constant;
 using OpenFeature.Error;
+using OpenFeature.Extension;
 using OpenFeature.Model;
 using OpenFeature.Tests.Internal;
 using Xunit;
@@ -365,5 +366,23 @@ public void Should_Get_And_Set_Context()
             client.SetContext(new EvaluationContextBuilder().Set(KEY, VAL).Build());
             Assert.Equal(VAL, client.GetContext().GetValue(KEY).AsInteger);
         }
+
+        [Fact]
+        public void ToFlagEvaluationDetails_Should_Convert_All_Properties()
+        {
+            var fixture = new Fixture();
+            var flagName = fixture.Create<string>();
+            var boolValue = fixture.Create<bool>();
+            var errorType = fixture.Create<ErrorType>();
+            var reason = fixture.Create<string>();
+            var variant  = fixture.Create<string>();
+            var errorMessage = fixture.Create<string>();
+            var flagMetadata = fixture.Create<FlagMetadata>();
+
+            var expected = new ResolutionDetails<bool>(flagName, boolValue, errorType, reason, variant, errorMessage, flagMetadata);
+            var result = expected.ToFlagEvaluationDetails();
+
+            result.Should().BeEquivalentTo(expected);
+        }
     }
 }