Update multiprovider name and FirstMatchStrategy to FirstFoundStrategy

Signed-off-by: jescriba <joshua.escribano@includedhealth.com>

Author: jescriba

Sources/OpenFeature/Provider/MultiProvider/FirstMatchStrategy.swift renamed to Sources/OpenFeature/Provider/MultiProvider/FirstFoundStrategy.swift

@@ -1,7 +1,7 @@
-/// FirstMatchStrategy is a strategy that evaluates a feature flag across multiple providers
+/// FirstFoundStrategy (FirstMatchStrategy) is a strategy that evaluates a feature flag across multiple providers
 /// and returns the first result. Skips providers that indicate they had no value due to flag not found.
 /// If any provider returns an error result other than flag not found, the error is returned.
-final public class FirstMatchStrategy: Strategy {
+final public class FirstFoundStrategy: Strategy {
     public init() {}
 
     public func evaluate<T>(

Sources/OpenFeature/Provider/MultiProvider/FirstSuccessfulStrategy.swift

@@ -1,5 +1,5 @@
 /// FirstSuccessfulStrategy is a strategy that evaluates a feature flag across multiple providers
-/// and returns the first result. Similar to `FirstMatchStrategy` but does not bubble up individual provider errors.
+/// and returns the first result. Similar to `FirstFoundStrategy` but does not bubble up individual provider errors.
 /// If no provider successfully responds, it will return an error.
 final public class FirstSuccessfulStrategy: Strategy {
     public func evaluate<T>(

Sources/OpenFeature/Provider/MultiProvider/MultiProvider.swift

@@ -8,21 +8,22 @@ public class MultiProvider: FeatureProvider {
     }
 
     public static let name = "MultiProvider"
-    public var metadata: ProviderMetadata = MultiProvider.MultiProviderMetadata()
+    public var metadata: ProviderMetadata
 
     private let providers: [FeatureProvider]
     private let strategy: Strategy
 
     /// Initialize a MultiProvider with a list of providers and a strategy.
     /// - Parameters:
     ///   - providers: A list of providers to evaluate.
-    ///   - strategy: A strategy to evaluate the providers. Defaults to FirstMatchStrategy.
+    ///   - strategy: A strategy to evaluate the providers. Defaults to FirstFoundStrategy.
     public init(
         providers: [FeatureProvider],
-        strategy: Strategy = FirstMatchStrategy()
+        strategy: Strategy = FirstFoundStrategy()
     ) {
         self.providers = providers
         self.strategy = strategy
+        self.metadata = MultiProviderMetadata(providers: providers)
     }
 
     public func initialize(initialContext: EvaluationContext?) async throws {
@@ -117,6 +118,12 @@ public class MultiProvider: FeatureProvider {
     }
 
     public struct MultiProviderMetadata: ProviderMetadata {
-        public var name: String? = MultiProvider.name
+        public var name: String?
+        
+        init(providers: [FeatureProvider]) {
+            self.name = providers.map({
+                $0.metadata.name ?? "MultiProvider"
+            }).joined(separator: ", ")
+        }
     }
 }

Tests/OpenFeatureTests/MultiProviderTests.swift

@@ -45,7 +45,7 @@ final class MultiProviderTests: XCTestCase {
         XCTAssertEqual(objectResult.value, mockProviderObjectValue)
     }
 
-    func testEvaluationWithMultipleProvidersAndFirstMatchStrategy_FirstProviderHasFlag() throws {
+    func testEvaluationWithMultipleProvidersAndFirstFoundStrategy_FirstProviderHasFlag() throws {
         let mockKey = "test-key"
         let mockProvider1Value = true
         let mockProvider1 = MockProvider(
@@ -58,15 +58,15 @@ final class MultiProviderTests: XCTestCase {
         )
         let multiProvider = MultiProvider(
             providers: [mockProvider1, mockProvider2],
-            strategy: FirstMatchStrategy()
+            strategy: FirstFoundStrategy()
         )
 
         let boolResult = try multiProvider.getBooleanEvaluation(
             key: mockKey, defaultValue: false, context: MutableContext())
         XCTAssertEqual(boolResult.value, mockProvider1Value)
     }
 
-    func testEvaluationWithMultipleProvidersAndFirstMatchStrategy_FlagNotFound() throws {
+    func testEvaluationWithMultipleProvidersAndFirstFoundStrategy_FlagNotFound() throws {
         let mockKey = "test-key"
         let mockProviderValue = true
         let mockProvider1 = MockProvider(
@@ -87,15 +87,15 @@ final class MultiProviderTests: XCTestCase {
         )
         let multiProvider = MultiProvider(
             providers: [mockProvider1, mockProvider2],
-            strategy: FirstMatchStrategy()
+            strategy: FirstFoundStrategy()
         )
 
         let boolResult = try multiProvider.getBooleanEvaluation(
             key: mockKey, defaultValue: false, context: MutableContext())
         XCTAssertEqual(boolResult.value, mockProviderValue)
     }
 
-    func testEvaluationWithMultipleProvidersAndFirstMatchStrategy_AllProvidersMissingFlag() throws {
+    func testEvaluationWithMultipleProvidersAndFirstFoundStrategy_AllProvidersMissingFlag() throws {
         let mockKey = "test-key"
         let mockProvider1 = MockProvider(
             initialize: { _ in },
@@ -109,7 +109,7 @@ final class MultiProviderTests: XCTestCase {
         )
         let multiProvider = MultiProvider(
             providers: [mockProvider1, mockProvider2],
-            strategy: FirstMatchStrategy()
+            strategy: FirstFoundStrategy()
         )
 
         let result = try multiProvider.getBooleanEvaluation(
@@ -120,7 +120,7 @@ final class MultiProviderTests: XCTestCase {
         XCTAssertTrue(result.errorCode == .flagNotFound)
     }
 
-    func testEvaluationWithMultipleProvidersAndFirstMatchStrategy_HandlesOpenFeatureError() throws {
+    func testEvaluationWithMultipleProvidersAndFirstFoundStrategy_HandlesOpenFeatureError() throws {
         let mockKey = "test-key"
         let mockProvider1 = MockProvider(
             initialize: { _ in },
@@ -136,7 +136,7 @@ final class MultiProviderTests: XCTestCase {
         )
         let multiProvider = MultiProvider(
             providers: [mockProvider1, mockProvider2],
-            strategy: FirstMatchStrategy()
+            strategy: FirstFoundStrategy()
         )
         let defaultValue = false
         let result = try multiProvider.getBooleanEvaluation(
@@ -145,7 +145,7 @@ final class MultiProviderTests: XCTestCase {
         XCTAssertNotNil(result.errorCode)
     }
 
-    func testEvaluationWithMultipleProvidersAndFirstMatchStrategy_Throws() throws {
+    func testEvaluationWithMultipleProvidersAndFirstFoundStrategy_Throws() throws {
         let mockKey = "test-key"
         let mockError = MockProvider.MockProviderError.message("test non-open feature error")
         let mockProvider1 = MockProvider(
@@ -162,7 +162,7 @@ final class MultiProviderTests: XCTestCase {
         )
         let multiProvider = MultiProvider(
             providers: [mockProvider1, mockProvider2],
-            strategy: FirstMatchStrategy()
+            strategy: FirstFoundStrategy()
         )
         let defaultValue = false
         do {