feat: Support retrieving numeric metadata as either either integers or decimals #443

Signed-off-by: Otto Lagerquist <[email protected]>
Signed-off-by: lager95 <[email protected]>

Author: lager95

src/OpenFeature/Model/ImmutableMetadata.cs

@@ -45,7 +45,14 @@ public ImmutableMetadata(Dictionary<string, object> metadata)
     /// <returns>The integer value associated with the key, or null if the key is not found.</returns>
     public int? GetInt(string key)
     {
-        return this.GetValue<int>(key);
+        
+        var hasValue = this._metadata.TryGetValue(key, out var value);
+        if (!hasValue)
+        {
+            return null;
+        }
+
+        return value is double || value is int ? Convert.ToInt32(value)  : null;
     }
 
     /// <summary>
@@ -55,7 +62,13 @@ public ImmutableMetadata(Dictionary<string, object> metadata)
     /// <returns>The double value associated with the key, or null if the key is not found.</returns>
     public double? GetDouble(string key)
     {
-        return this.GetValue<double>(key);
+        var hasValue = this._metadata.TryGetValue(key, out var value);
+        if (!hasValue)
+        {
+            return null;
+        }
+
+        return value is double || value is int ? Convert.ToDouble(value)  : null;
     }
 
     /// <summary>

test/OpenFeature.Tests/ImmutableMetadataTest.cs

@@ -100,6 +100,28 @@ public void GetInt_Should_Return_Value_If_Key_Found()
         Assert.NotNull(result);
         Assert.Equal(1, result);
     }
+    [Fact]
+    [Specification("1.4.14",
+        "If the `flag metadata` field in the `flag resolution` structure returned by the configured `provider` is set, the `evaluation details` structure's `flag metadata` field MUST contain that value. Otherwise, it MUST contain an empty record.")]
+    [Specification("1.4.14.1", "Condition: `Flag metadata` MUST be immutable.")]
+    public void GetInt_Should_Return_Value_If_Key_Found_although_double()
+    {
+        // Arrange
+        var metadata = new Dictionary<string, object>
+        {
+            {
+                "intKey", 1.0
+            }
+        };
+        var flagMetadata = new ImmutableMetadata(metadata);
+
+        // Act
+        var result = flagMetadata.GetInt("intKey");
+
+        // Assert
+        Assert.NotNull(result);
+        Assert.Equal(1, result);
+    }
 
     [Fact]
     [Specification("1.4.14",
@@ -160,6 +182,29 @@ public void GetDouble_Should_Return_Value_If_Key_Found()
         Assert.Equal(1.2, result);
     }
 
+    [Fact]
+    [Specification("1.4.14",
+        "If the `flag metadata` field in the `flag resolution` structure returned by the configured `provider` is set, the `evaluation details` structure's `flag metadata` field MUST contain that value. Otherwise, it MUST contain an empty record.")]
+    [Specification("1.4.14.1", "Condition: `Flag metadata` MUST be immutable.")]
+    public void GetDouble_Should_Return_Value_If_Key_Found_Although_Int()
+    {
+        // Arrange
+        var metadata = new Dictionary<string, object>
+        {
+            {
+                "doubleKey", 1
+            }
+        };
+        var flagMetadata = new ImmutableMetadata(metadata);
+
+        // Act
+        var result = flagMetadata.GetDouble("doubleKey");
+
+        // Assert
+        Assert.NotNull(result);
+        Assert.Equal(1.0, result);
+    }
+
     [Fact]
     [Specification("1.4.14",
         "If the `flag metadata` field in the `flag resolution` structure returned by the configured `provider` is set, the `evaluation details` structure's `flag metadata` field MUST contain that value. Otherwise, it MUST contain an empty record.")]