feat: add evaluation metadata to evaluation details

Signed-off-by: Nicklas Lundin <[email protected]>

Author: nicklasl

android/src/main/java/dev/openfeature/sdk/FlagEvaluationDetails.kt

@@ -8,7 +8,8 @@ data class FlagEvaluationDetails<T>(
     val variant: String? = null,
     val reason: String? = null,
     val errorCode: ErrorCode? = null,
-    val errorMessage: String? = null
+    val errorMessage: String? = null,
+    val metadata: EvaluationMetadata = EvaluationMetadata.EMPTY
 ) {
     companion object
 }
@@ -18,11 +19,12 @@ fun <T> FlagEvaluationDetails.Companion.from(
     flagKey: String
 ): FlagEvaluationDetails<T> {
     return FlagEvaluationDetails(
-        flagKey,
-        providerEval.value,
-        providerEval.variant,
-        providerEval.reason,
-        providerEval.errorCode,
-        providerEval.errorMessage
+        flagKey = flagKey,
+        value = providerEval.value,
+        variant = providerEval.variant,
+        reason = providerEval.reason,
+        errorCode = providerEval.errorCode,
+        errorMessage = providerEval.errorMessage,
+        metadata = providerEval.metadata
     )
 }

android/src/main/java/dev/openfeature/sdk/FlagEvaluationMetadata.kt

@@ -16,6 +16,8 @@ class EvaluationMetadata internal constructor(private val values: Map<String, An
         fun builder(): Builder {
             return Builder()
         }
+
+        val EMPTY = EvaluationMetadata(emptyMap())
     }
 
     override fun equals(other: Any?): Boolean {

android/src/main/java/dev/openfeature/sdk/ProviderEvaluation.kt

@@ -7,5 +7,6 @@ data class ProviderEvaluation<T>(
     val variant: String? = null,
     val reason: String? = null,
     val errorCode: ErrorCode? = null,
-    val errorMessage: String? = null
+    val errorMessage: String? = null,
+    val metadata: EvaluationMetadata = EvaluationMetadata.EMPTY
 )

android/src/test/java/dev/openfeature/sdk/FlagEvaluationsTests.kt

@@ -76,7 +76,8 @@ class FlagEvaluationsTests {
         Assert.assertEquals(booleanDetails, client.getBooleanDetails(key, false))
         Assert.assertEquals(booleanDetails, client.getBooleanDetails(key, false, FlagEvaluationOptions()))
 
-        val stringDetails = FlagEvaluationDetails(key, "tset")
+        // in DoSomethingProvider, the string evaluation is special since it contains some metadata values
+        val stringDetails = FlagEvaluationDetails(key, "tset", metadata = DoSomethingProvider.evaluationMetadata)
         Assert.assertEquals(stringDetails, client.getStringDetails(key, "test"))
         Assert.assertEquals(stringDetails, client.getStringDetails(key, "test", FlagEvaluationOptions()))
 
@@ -93,6 +94,18 @@ class FlagEvaluationsTests {
         Assert.assertEquals(objectDetails, client.getObjectDetails(key, Value.Structure(mapOf()), FlagEvaluationOptions()))
     }
 
+    @Test
+    fun testMetadataFlagEvaluation() = runTest {
+        OpenFeatureAPI.setProvider(DoSomethingProvider())
+        val client = OpenFeatureAPI.getClient()
+        val key = "key"
+
+        val details = client.getStringDetails(key, "default")
+        val metadata: EvaluationMetadata = details.metadata
+        Assert.assertEquals("value1", metadata.getString("key1"))
+        Assert.assertEquals(42, metadata.getInt("key2"))
+    }
+
     @Test
     fun testHooksAreFired() = runTest {
         OpenFeatureAPI.setProvider(NoOpProvider())

android/src/test/java/dev/openfeature/sdk/helpers/DoSomethingProvider.kt

@@ -1,6 +1,7 @@
 package dev.openfeature.sdk.helpers
 
 import dev.openfeature.sdk.EvaluationContext
+import dev.openfeature.sdk.EvaluationMetadata
 import dev.openfeature.sdk.FeatureProvider
 import dev.openfeature.sdk.Hook
 import dev.openfeature.sdk.ProviderEvaluation
@@ -19,6 +20,12 @@ class DoSomethingProvider(
     override val metadata: ProviderMetadata = DoSomethingProviderMetadata(),
     private var dispatcher: CoroutineDispatcher = Dispatchers.IO
 ) : FeatureProvider {
+    companion object {
+        val evaluationMetadata = EvaluationMetadata.builder()
+            .putString("key1", "value1")
+            .putInt("key2", 42)
+            .build()
+    }
     private var eventHandler = EventHandler(dispatcher)
 
     override fun initialize(initialContext: EvaluationContext?) {
@@ -51,7 +58,10 @@ class DoSomethingProvider(
         defaultValue: String,
         context: EvaluationContext?
     ): ProviderEvaluation<String> {
-        return ProviderEvaluation(defaultValue.reversed())
+        return ProviderEvaluation(
+            value = defaultValue.reversed(),
+            metadata = evaluationMetadata
+        )
     }
 
     override fun getIntegerEvaluation(