Release 1.6.3 (#94)

* Release 1.6.3

* fix(fs/schema): only run validation if field is present (#93)

Author: gagantrivedi

flag_engine/features/schemas.py

@@ -69,6 +69,11 @@ def make_feature_state(self, data, **kwargs) -> FeatureStateModel:
     def validate_percentage_allocations(self, data, **kwargs):
         """Since we do support modifying percentage allocation on a per identity override bases
         we need to validate the percentage before building the document(dict)"""
+        # Since client(s) can exclude this field from dumping we need to make sure
+        # We only run the validation if the field is present on the serializer
+        if "multivariate_feature_state_values" not in self.fields:
+            return data
+
         total_allocation = sum(
             mvfsv["percentage_allocation"]
             for mvfsv in data["multivariate_feature_state_values"]

setup.py

@@ -2,7 +2,7 @@
 
 setup(
     name="flagsmith-flag-engine",
-    version="1.6.2",
+    version="1.6.3",
     author="Flagsmith",
     author_email="[email protected]",
     packages=find_packages(include=["flag_engine", "flag_engine.*"]),

tests/unit/features/test_schemas.py

@@ -46,3 +46,14 @@ def test_dumping_fs_schema_works_for_valid_allocation():
     }
     # Then
     FeatureStateSchema().dump(data)
+
+
+def test_can_dump_featurestate_schema_without_mvfs(feature_1):
+    # Given
+    raw_data = {"feature_state_value": "value"}
+    # When
+    dumped_data = FeatureStateSchema(
+        exclude=["multivariate_feature_state_values"]
+    ).dump(raw_data)
+    # Then
+    assert raw_data["feature_state_value"] == dumped_data["feature_state_value"]