growthbook 2.1.1__tar.gz → 2.1.3__tar.gz

{growthbook-2.1.1/growthbook.egg-info → growthbook-2.1.3}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: growthbook
-Version: 2.1.1
+Version: 2.1.3
 Summary: Powerful Feature flagging and A/B testing for Python apps
 Home-page: https://github.com/growthbook/growthbook-python
 Author: GrowthBook

{growthbook-2.1.1 → growthbook-2.1.3}/growthbook/__init__.py

@@ -18,5 +18,5 @@ from .plugins import (
 )
 
 # x-release-please-start-version
-__version__ = "2.1.1"
+__version__ = "2.1.3"
 # x-release-please-end

{growthbook-2.1.1 → growthbook-2.1.3}/growthbook/core.py

@@ -50,47 +50,48 @@ def isOperatorObject(obj: Any) -> bool:
             return False
     return True
 
-def getType(attributeValue) -> str:
-    t = type(attributeValue)
+def _is_numeric(v: Any) -> bool:
+    return isinstance(v, (int, float)) and not isinstance(v, bool)
 
+def getType(attributeValue) -> str:
     if attributeValue is None:
         return "null"
-    if t is int or t is float:
+    if isinstance(attributeValue, bool):
+        return "boolean"
+    if _is_numeric(attributeValue):
         return "number"
-    if t is str:
+    if isinstance(attributeValue, str):
         return "string"
-    if t is list or t is set:
+    if isinstance(attributeValue, (list, set)):
         return "array"
-    if t is dict:
+    if isinstance(attributeValue, dict):
         return "object"
-    if t is bool:
-        return "boolean"
     return "unknown"
 
 def getPath(attributes, path):
     current = attributes
     for segment in path.split("."):
-        if type(current) is dict and segment in current:
+        if isinstance(current, dict) and segment in current:
             current = current[segment]
         else:
             return None
     return current
 
 def evalConditionValue(conditionValue, attributeValue, savedGroups, insensitive: bool = False) -> bool:
-    if type(conditionValue) is dict and isOperatorObject(conditionValue):
+    if isinstance(conditionValue, dict) and isOperatorObject(conditionValue):
         for key, value in conditionValue.items():
             if not evalOperatorCondition(key, attributeValue, value, savedGroups):
                 return False
         return True
     
     # Simple equality comparison with optional case-insensitivity
-    if insensitive and type(conditionValue) is str and type(attributeValue) is str:
+    if insensitive and isinstance(conditionValue, str) and isinstance(attributeValue, str):
         return conditionValue.lower() == attributeValue.lower()
     
     return bool(conditionValue == attributeValue)
 
 def elemMatch(condition, attributeValue, savedGroups) -> bool:
-    if not type(attributeValue) is list:
+    if not isinstance(attributeValue, list):
         return False
 
     for item in attributeValue:
@@ -104,13 +105,13 @@ def elemMatch(condition, attributeValue, savedGroups) -> bool:
     return False
 
 def compare(val1, val2) -> int:
-    if (type(val1) is int or type(val1) is float) and not (type(val2) is int or type(val2) is float):
+    if _is_numeric(val1) and not _is_numeric(val2):
         if (val2 is None):
             val2 = 0
         else:
             val2 = float(val2)
 
-    if (type(val2) is int or type(val2) is float) and not (type(val1) is int or type(val1) is float):
+    if _is_numeric(val2) and not _is_numeric(val1):
         if (val1 is None):
             val1 = 0
         else:
@@ -166,13 +167,13 @@ def evalOperatorCondition(operator, attributeValue, conditionValue, savedGroups)
     elif operator == "$vgte":
         return paddedVersionString(attributeValue) >= paddedVersionString(conditionValue)
     elif operator == "$inGroup":
-        if not type(conditionValue) is str:
+        if not isinstance(conditionValue, str):
             return False
         if not conditionValue in savedGroups:
             return False
         return isIn(savedGroups[conditionValue] or [], attributeValue)
     elif operator == "$notInGroup":
-        if not type(conditionValue) is str:
+        if not isinstance(conditionValue, str):
             return False
         if not conditionValue in savedGroups:
             return True
@@ -202,33 +203,33 @@ def evalOperatorCondition(operator, attributeValue, conditionValue, savedGroups)
         except Exception:
             return False
     elif operator == "$in":
-        if not type(conditionValue) is list:
+        if not isinstance(conditionValue, list):
             return False
         return isIn(conditionValue, attributeValue)
     elif operator == "$nin":
-        if not type(conditionValue) is list:
+        if not isinstance(conditionValue, list):
             return False
         return not isIn(conditionValue, attributeValue)
     elif operator == "$ini":
-        if not type(conditionValue) is list:
+        if not isinstance(conditionValue, list):
             return False
         return isIn(conditionValue, attributeValue, insensitive=True)
     elif operator == "$nini":
-        if not type(conditionValue) is list:
+        if not isinstance(conditionValue, list):
             return False
         return not isIn(conditionValue, attributeValue, insensitive=True)
     elif operator == "$elemMatch":
         return elemMatch(conditionValue, attributeValue, savedGroups)
     elif operator == "$size":
-        if not (type(attributeValue) is list):
+        if not isinstance(attributeValue, list):
             return False
         return evalConditionValue(conditionValue, len(attributeValue), savedGroups)
     elif operator == "$all":
-        if not type(conditionValue) is list:
+        if not isinstance(conditionValue, list):
             return False
         return isInAll(conditionValue, attributeValue, savedGroups, insensitive=False)
     elif operator == "$alli":
-        if not type(conditionValue) is list:
+        if not isinstance(conditionValue, list):
             return False
         return isInAll(conditionValue, attributeValue, savedGroups, insensitive=True)
     elif operator == "$exists":
@@ -243,10 +244,10 @@ def evalOperatorCondition(operator, attributeValue, conditionValue, savedGroups)
 
 def paddedVersionString(input) -> str:
     # If input is a number, convert to a string
-    if type(input) is int or type(input) is float:
+    if _is_numeric(input):
         input = str(input)
 
-    if not input or type(input) is not str:
+    if not input or not isinstance(input, str):
         input = "0"
 
     # Remove build info and leading `v` if any
@@ -268,10 +269,10 @@ def isIn(conditionValue, attributeValue, insensitive: bool = False) -> bool:
     if insensitive:
         # Helper function to case-fold values (lowercase for strings)
         def case_fold(val):
-            return val.lower() if type(val) is str else val
+            return val.lower() if isinstance(val, str) else val
         
         # Do an intersection if attribute is an array (insensitive)
-        if type(attributeValue) is list:
+        if isinstance(attributeValue, list):
             return any(
                 case_fold(el) == case_fold(exp)
                 for el in attributeValue
@@ -280,13 +281,13 @@ def isIn(conditionValue, attributeValue, insensitive: bool = False) -> bool:
         return any(case_fold(attributeValue) == case_fold(exp) for exp in conditionValue)
     
     # Case-sensitive behavior (original)
-    if type(attributeValue) is list:
+    if isinstance(attributeValue, list):
         return bool(set(conditionValue) & set(attributeValue))
     return attributeValue in conditionValue
 
 def isInAll(conditionValue, attributeValue, savedGroups, insensitive: bool = False) -> bool:
     """Check if attributeValue (array) contains all elements in conditionValue"""
-    if not type(attributeValue) is list:
+    if not isinstance(attributeValue, list):
         return False
     
     for cond in conditionValue:

{growthbook-2.1.1 → growthbook-2.1.3}/growthbook/growthbook_client.py

@@ -90,8 +90,8 @@ class FeatureCache:
     def update(self, features: Dict[str, Any], saved_groups: Dict[str, Any]) -> None:
         """Simple thread-safe update of cache with new API data"""
         with self._lock:
-            self._cache['features'].update(features)
-            self._cache['savedGroups'].update(saved_groups)
+            self._cache['features'] = dict(features)
+            self._cache['savedGroups'] = dict(saved_groups)
 
     def get_current_state(self) -> Dict[str, Any]:
         """Get current cache state"""

{growthbook-2.1.1 → growthbook-2.1.3/growthbook.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: growthbook
-Version: 2.1.1
+Version: 2.1.3
 Summary: Powerful Feature flagging and A/B testing for Python apps
 Home-page: https://github.com/growthbook/growthbook-python
 Author: GrowthBook

{growthbook-2.1.1 → growthbook-2.1.3}/growthbook.egg-info/SOURCES.txt

@@ -20,6 +20,7 @@ growthbook/plugins/base.py
 growthbook/plugins/growthbook_tracking.py
 growthbook/plugins/request_context.py
 tests/conftest.py
+tests/test_dict_subclass.py
 tests/test_etag.py
 tests/test_growthbook.py
 tests/test_growthbook_client.py

{growthbook-2.1.1 → growthbook-2.1.3}/setup.cfg

@@ -1,5 +1,5 @@
 [bumpversion]
-current_version = 2.1.1
+current_version = 2.1.3
 commit = True
 tag = True
 

growthbook-2.1.3/tests/test_dict_subclass.py

@@ -0,0 +1,33 @@
+import unittest
+from growthbook.core import getPath, evalCondition
+
+class MyDict(dict):
+    pass
+
+class TestDictSubclass(unittest.TestCase):
+    def test_get_path_with_subclass(self):
+        # Test getPath with a dict subclass
+        attributes = MyDict({"user": MyDict({"id": "123", "name": "John"})})
+        
+        self.assertEqual(getPath(attributes, "user.id"), "123")
+        self.assertEqual(getPath(attributes, "user.name"), "John")
+        self.assertEqual(getPath(attributes, "user.nonexistent"), None)
+
+    def test_eval_condition_with_subclass(self):
+        # Test evalCondition with a dict subclass
+        attributes = MyDict({"company": "GrowthBook", "meta": MyDict({"plan": "pro"})})
+        
+        # Simple condition
+        condition = {"company": "GrowthBook"}
+        self.assertTrue(evalCondition(attributes, condition))
+        
+        # Nested condition using getPath (indirectly)
+        condition = {"meta.plan": "pro"}
+        self.assertTrue(evalCondition(attributes, condition))
+        
+        # Condition failing
+        condition = {"meta.plan": "free"}
+        self.assertFalse(evalCondition(attributes, condition))
+
+if __name__ == '__main__':
+    unittest.main()

{growthbook-2.1.1 → growthbook-2.1.3}/tests/test_growthbook_client.py

@@ -158,19 +158,93 @@ async def test_concurrent_feature_updates():
         client_key="test_key"
     )
     features = {f"feature-{i}": {"defaultValue": i} for i in range(10)}
-    
+
     async def update_features(feature_subset):
         await repo._handle_feature_update({"features": feature_subset, "savedGroups": {}})
-            
+
     await asyncio.gather(*[
-        update_features({k: features[k]}) 
+        update_features({k: features[k]})
         for k in features
     ])
-    
+
     cache_state = repo._feature_cache.get_current_state()
     # Verify all features were properly stored
-    assert cache_state["features"] == features
+    assert len(cache_state["features"]) == 1
     assert cache_state["savedGroups"] == {}
+    feature_key = list(cache_state["features"].keys())[0]
+    assert feature_key in features
+    assert cache_state["features"][feature_key] == features[feature_key]
+
+
+@pytest.mark.asyncio
+async def test_feature_cache_thread_safety(cache):
+    """Verify FeatureCache is thread-safe during concurrent updates"""
+    repo = EnhancedFeatureRepository(
+        api_host="https://test.growthbook.io",
+        client_key="test_key"
+    )
+
+    feature_sets = [
+        {f"set-{i}-feature-{j}": {"value": j} for j in range(3)}
+        for i in range(5)
+    ]
+
+    async def update_full_set(feature_set):
+        await repo._handle_feature_update({
+            "features": feature_set,
+            "savedGroups": {}
+        })
+
+    # Concurrent updates
+    await asyncio.gather(*[update_full_set(fs) for fs in feature_sets])
+
+    cache = repo._feature_cache.get_current_state()
+
+    # One complete set should be in cache (race condition winner)
+    assert len(cache["features"]) == 3
+    cache_keys = set(cache["features"].keys())
+    assert any(cache_keys == set(fs.keys()) for fs in feature_sets)
+
+
+@pytest.mark.asyncio
+async def test_disabled_features_removed_from_cache(cache):
+    """
+    Regression test: disabled features must be removed from cache.
+
+    Previously, FeatureCache.update() used dict.update() which only
+    adds/modifies entries but never removes them. This caused disabled
+    features to persist in the cache indefinitely.
+    """
+    repo = EnhancedFeatureRepository(
+        api_host="https://test.growthbook.io",
+        client_key="test_key"
+    )
+
+    # Initial state: 2 features enabled
+    await repo._handle_feature_update({
+        "features": {
+            "feature-a": {"defaultValue": True},
+            "feature-b": {"defaultValue": False}
+        },
+        "savedGroups": {}
+    })
+
+    cache = repo._feature_cache.get_current_state()
+    assert "feature-a" in cache["features"]
+    assert "feature-b" in cache["features"]
+
+    # User disables feature-b in Growthbook UI
+    # API now returns only active features
+    await repo._handle_feature_update({
+        "features": {
+            "feature-a": {"defaultValue": True}
+        },
+        "savedGroups": {}
+    })
+
+    cache = repo._feature_cache.get_current_state()
+    assert "feature-a" in cache["features"]
+    assert "feature-b" not in cache["features"]  # Must be removed!
 
 @pytest.mark.asyncio
 async def test_callback_thread_safety():