Remove context on child instance

spotify · Apr 9, 2024 · 560f09b · 560f09b
1 parent a331d76
commit 560f09b
Show file tree

Hide file tree

Showing 3 changed files with 107 additions and 13 deletions.
diff --git a/Sources/Confidence/Confidence.swift b/Sources/Confidence/Confidence.swift
@@ -7,6 +7,7 @@ public class Confidence: ConfidenceEventSender {
     public var timeout: TimeInterval
     public var region: ConfidenceRegion
     public var initializationStrategy: InitializationStrategy
+    private var removedContextKeys: Set<String> = Set()
 
     required public init(
         clientSecret: String,
@@ -31,22 +32,28 @@ public class Confidence: ConfidenceEventSender {
 
 
     public func getContext() -> ConfidenceStruct {
-        var mergedContext = parent?.getContext() ?? [:]
+        let parentContext = parent?.getContext() ?? [:]
+        var reconciledCtx = parentContext.filter {
+            !removedContextKeys.contains($0.key)
+        }
         self.context.forEach { entry in
-            mergedContext.updateValue(entry.value, forKey: entry.key)
+            reconciledCtx.updateValue(entry.value, forKey: entry.key)
         }
-        return mergedContext
+        return reconciledCtx
     }
 
     public func updateContextEntry(key: String, value: ConfidenceValue) {
         context[key] = value
+        removedContextKeys.remove(key)
     }
 
     public func removeContextEntry(key: String) {
         context.removeValue(forKey: key)
+        removedContextKeys.insert(key)
     }
 
     public func clearContext() {
+        removedContextKeys.removeAll()
         context = [:]
     }
 

diff --git a/Sources/Confidence/Contextual.swift b/Sources/Confidence/Contextual.swift
@@ -1,12 +1,16 @@
 import Foundation
 
-/// A Contextual implementer maintains context data and can create child instances
+/// A Contextual implementer maintains local context data and can create child instances
 /// that can still access their parent's data
+/// Each ConfidenceContextProvider returns local data reconciled with parents' data. Local data has precedence
 public protocol Contextual: ConfidenceContextProvider {
+    /// Adds entry to local data
     func updateContextEntry(key: String, value: ConfidenceValue)
+    /// Removes entry from local data
+    /// It hides entries with this key from parents' data (without modifying parents' data)
     func removeContextEntry(key: String)
+    /// Clear local data. Parents' data is still reconciled. Previously removed parent entries are not hidden anymore
     func clearContext()
-    /// Creates a child Contextual instance that still has access
-    /// to its parent context
+    /// Creates a child Contextual instance that maintains access to its parent's data
     func withContext(_ context: ConfidenceStruct) -> Self
 }
diff --git a/Tests/ConfidenceTests/ConfidenceTests.swift b/Tests/ConfidenceTests/ConfidenceTests.swift
@@ -42,7 +42,7 @@ final class ConfidenceTests: XCTestCase {
         XCTAssertEqual(confidenceChild.getContext(), expected)
     }
 
-    func testUpdateContextWithOverride() {
+    func testUpdateLocalContext() {
         let confidence = Confidence.init(
             clientSecret: "",
             timeout: TimeInterval(),
@@ -59,7 +59,7 @@ final class ConfidenceTests: XCTestCase {
         XCTAssertEqual(confidence.getContext(), expected)
     }
 
-    func testWithContextUpdateParentWithoutOverride() {
+    func testUpdateLocalContextWithoutOverride() {
         let confidenceParent = Confidence.init(
             clientSecret: "",
             timeout: TimeInterval(),
@@ -70,17 +70,17 @@ final class ConfidenceTests: XCTestCase {
         let confidenceChild: ConfidenceEventSender = confidenceParent.withContext(
             ["k2": ConfidenceValue(string: "v2")]
         )
-        confidenceParent.updateContextEntry(
+        confidenceChild.updateContextEntry(
             key: "k2",
             value: ConfidenceValue(string: "v4"))
         let expected = [
             "k1": ConfidenceValue(string: "v1"),
-            "k2": ConfidenceValue(string: "v2"),
+            "k2": ConfidenceValue(string: "v4"),
         ]
         XCTAssertEqual(confidenceChild.getContext(), expected)
     }
 
-    func testWithContextUpdateChildWithOverride() {
+    func testUpdateParentContextWithOverride() {
         let confidenceParent = Confidence.init(
             clientSecret: "",
             timeout: TimeInterval(),
@@ -91,12 +91,12 @@ final class ConfidenceTests: XCTestCase {
         let confidenceChild: ConfidenceEventSender = confidenceParent.withContext(
             ["k2": ConfidenceValue(string: "v2")]
         )
-        confidenceChild.updateContextEntry(
+        confidenceParent.updateContextEntry(
             key: "k2",
             value: ConfidenceValue(string: "v4"))
         let expected = [
             "k1": ConfidenceValue(string: "v1"),
-            "k2": ConfidenceValue(string: "v4"),
+            "k2": ConfidenceValue(string: "v2"),
         ]
         XCTAssertEqual(confidenceChild.getContext(), expected)
     }
@@ -119,6 +119,68 @@ final class ConfidenceTests: XCTestCase {
         XCTAssertEqual(confidence.getContext(), expected)
     }
 
+    func testRemoveContextEntryFromParent() {
+        let confidenceParent = Confidence.init(
+            clientSecret: "",
+            timeout: TimeInterval(),
+            region: .europe,
+            initializationStrategy: .activateAndFetchAsync,
+            context: ["k1": ConfidenceValue(string: "v1")]
+        )
+        let confidenceChild: ConfidenceEventSender = confidenceParent.withContext(
+            ["k2": ConfidenceValue(string: "v2")]
+        )
+        confidenceChild.removeContextEntry(key: "k1")
+        let expected = [
+            "k2": ConfidenceValue(string: "v2")
+        ]
+        XCTAssertEqual(confidenceChild.getContext(), expected)
+    }
+
+    func testRemoveContextEntryFromParentAndChild() {
+        let confidenceParent = Confidence.init(
+            clientSecret: "",
+            timeout: TimeInterval(),
+            region: .europe,
+            initializationStrategy: .activateAndFetchAsync,
+            context: ["k1": ConfidenceValue(string: "v1")]
+        )
+        let confidenceChild: ConfidenceEventSender = confidenceParent.withContext(
+            [
+                "k2": ConfidenceValue(string: "v2"),
+                "k1": ConfidenceValue(string: "v3"),
+            ]
+        )
+        confidenceChild.removeContextEntry(key: "k1")
+        let expected = [
+            "k2": ConfidenceValue(string: "v2")
+        ]
+        XCTAssertEqual(confidenceChild.getContext(), expected)
+    }
+
+    func testRemoveContextEntryFromParentAndChildThenUpdate() {
+        let confidenceParent = Confidence.init(
+            clientSecret: "",
+            timeout: TimeInterval(),
+            region: .europe,
+            initializationStrategy: .activateAndFetchAsync,
+            context: ["k1": ConfidenceValue(string: "v1")]
+        )
+        let confidenceChild: ConfidenceEventSender = confidenceParent.withContext(
+            [
+                "k2": ConfidenceValue(string: "v2"),
+                "k1": ConfidenceValue(string: "v3"),
+            ]
+        )
+        confidenceChild.removeContextEntry(key: "k1")
+        confidenceChild.updateContextEntry(key: "k1", value: ConfidenceValue(string: "v4"))
+        let expected = [
+            "k2": ConfidenceValue(string: "v2"),
+            "k1": ConfidenceValue(string: "v4"),
+        ]
+        XCTAssertEqual(confidenceChild.getContext(), expected)
+    }
+
     func testClearContext() {
         let confidence = Confidence.init(
             clientSecret: "",
@@ -134,4 +196,25 @@ final class ConfidenceTests: XCTestCase {
         let expected: ConfidenceStruct = [:]
         XCTAssertEqual(confidence.getContext(), expected)
     }
+
+    func testClearContextReturnsParentContext() {
+        let confidenceParent = Confidence.init(
+            clientSecret: "",
+            timeout: TimeInterval(),
+            region: .europe,
+            initializationStrategy: .activateAndFetchAsync,
+            context: ["k1": ConfidenceValue(string: "v1")]
+        )
+        let confidenceChild: ConfidenceEventSender = confidenceParent.withContext(
+            [
+                "k1": ConfidenceValue(string: "v1"),
+                "k2": ConfidenceValue(string: "v2")
+            ]
+        )
+        confidenceChild.clearContext()
+        let expected = [
+            "k1": ConfidenceValue(string: "v1")
+        ]
+        XCTAssertEqual(confidenceChild.getContext(), expected)
+    }
 }