Merge pull request #135 from Flagsmith/release/2.1.0

Release 2.1.0

Author: matthewelwell

flag_engine/api/schemas.py

@@ -10,7 +10,6 @@
 from flag_engine.environments.schemas import (
     BaseEnvironmentAPIKeySchema,
     BaseEnvironmentSchema,
-    WebhookSchema,
 )
 from flag_engine.features.schemas import (
     BaseFeatureStateSchema,
@@ -140,7 +139,6 @@ class DjangoEnvironmentSchema(BaseEnvironmentSchema):
         dump_only=True,
     )
     project = fields.Nested(DjangoProjectSchema, dump_only=True)
-    webhooks = DjangoRelatedManagerField(fields.Nested(WebhookSchema), required=False)
 
     @pre_dump()
     def set_environment_key_in_context(self, obj, *args, **kwargs):

flag_engine/environments/models.py

@@ -27,11 +27,8 @@ def is_valid(self):
 
 @dataclass
 class WebhookModel:
-    created_at: datetime
-    updated_at: datetime
     url: str
     secret: str
-    enabled: bool = True
 
 
 @dataclass
@@ -45,7 +42,7 @@ class EnvironmentModel:
     mixpanel_config: IntegrationModel = None
     heap_config: IntegrationModel = None
     dynatrace_config: IntegrationModel = None
-    webhooks: typing.List[WebhookModel] = field(default_factory=list)
+    webhook_config: WebhookModel = None
 
     _INTEGRATION_ATTS = [
         "amplitude_config",

flag_engine/environments/schemas.py

@@ -28,11 +28,8 @@ class Meta:
 
 
 class WebhookSchema(LoadToModelSchema):
-    enabled = fields.Bool()
     url = fields.URL()
     secret = fields.Str()
-    created_at = fields.DateTime()
-    updated_at = fields.DateTime()
 
     class Meta:
         model_class = WebhookModel
@@ -46,14 +43,12 @@ class BaseEnvironmentSchema(Schema):
     mixpanel_config = fields.Nested(IntegrationSchema, required=False, allow_none=True)
     amplitude_config = fields.Nested(IntegrationSchema, required=False, allow_none=True)
     dynatrace_config = fields.Nested(IntegrationSchema, required=False, allow_none=True)
+    webhook_config = fields.Nested(WebhookSchema, required=False, allow_none=True)
 
 
 class EnvironmentSchema(LoadToModelMixin, BaseEnvironmentSchema):
     feature_states = fields.List(fields.Nested(FeatureStateSchema))
     project = fields.Nested(ProjectSchema)
-    webhooks = fields.List(
-        fields.Nested(WebhookSchema), required=False, allow_none=True
-    )
 
     class Meta:
         model_class = EnvironmentModel

flag_engine/segments/models.py

@@ -1,5 +1,6 @@
 import re
 import typing
+from contextlib import suppress
 from dataclasses import dataclass, field
 
 import semver
@@ -23,28 +24,31 @@ class SegmentConditionModel:
 
     def matches_trait_value(self, trait_value: typing.Any) -> bool:
         # TODO: move this logic to the evaluator module
-        if type(self.value) is str and is_semver(self.value):
-            trait_value = semver.VersionInfo.parse(trait_value)
-        if self.operator in self.EXCEPTION_OPERATOR_METHODS:
-            evaluator_function = getattr(
-                self, self.EXCEPTION_OPERATOR_METHODS.get(self.operator)
+        with suppress(ValueError):
+            if type(self.value) is str and is_semver(self.value):
+                trait_value = semver.VersionInfo.parse(trait_value)
+            if self.operator in self.EXCEPTION_OPERATOR_METHODS:
+                evaluator_function = getattr(
+                    self, self.EXCEPTION_OPERATOR_METHODS.get(self.operator)
+                )
+                return evaluator_function(trait_value)
+
+            matching_function_name = {
+                constants.EQUAL: "__eq__",
+                constants.GREATER_THAN: "__gt__",
+                constants.GREATER_THAN_INCLUSIVE: "__ge__",
+                constants.LESS_THAN: "__lt__",
+                constants.LESS_THAN_INCLUSIVE: "__le__",
+                constants.NOT_EQUAL: "__ne__",
+                constants.CONTAINS: "__contains__",
+            }.get(self.operator)
+            matching_function = getattr(
+                trait_value, matching_function_name, lambda v: False
             )
-            return evaluator_function(trait_value)
-
-        matching_function_name = {
-            constants.EQUAL: "__eq__",
-            constants.GREATER_THAN: "__gt__",
-            constants.GREATER_THAN_INCLUSIVE: "__ge__",
-            constants.LESS_THAN: "__lt__",
-            constants.LESS_THAN_INCLUSIVE: "__le__",
-            constants.NOT_EQUAL: "__ne__",
-            constants.CONTAINS: "__contains__",
-        }.get(self.operator)
-        matching_function = getattr(
-            trait_value, matching_function_name, lambda v: False
-        )
-        to_same_type_as_trait_value = get_casting_function(trait_value)
-        return matching_function(to_same_type_as_trait_value(self.value))
+            to_same_type_as_trait_value = get_casting_function(trait_value)
+            return matching_function(to_same_type_as_trait_value(self.value))
+
+        return False
 
     def evaluate_not_contains(self, trait_value: typing.Iterable) -> bool:
         return self.value not in trait_value

setup.py

@@ -2,7 +2,7 @@
 
 setup(
     name="flagsmith-flag-engine",
-    version="2.0.7",
+    version="2.1.0",
     author="Flagsmith",
     author_email="support@flagsmith.com",
     packages=find_packages(include=["flag_engine", "flag_engine.*"]),

tests/mock_django_classes.py

@@ -204,20 +204,9 @@ def all(self) -> typing.List[DjangoFeatureState]:
 
 
 @dataclass
-class DjangoWebhook:
+class DjangoWebhookConfig:
     url: str
     secret: str
-    created_at: datetime = field(default_factory=utcnow_with_tz)
-    updated_at: datetime = field(default_factory=utcnow_with_tz)
-    enabled: bool = True
-
-
-@dataclass
-class DjangoWebhookRelatedManager:
-    webhooks: typing.List[DjangoWebhook]
-
-    def all(self) -> typing.List[DjangoWebhook]:
-        return self.webhooks
 
 
 class DjangoEnvironment:
@@ -228,7 +217,7 @@ def __init__(
         name: str = "Test Environment",
         api_key: str = "api-key",
         feature_states: typing.List[DjangoFeatureState] = None,
-        webhooks: typing.List[DjangoWebhook] = None,
+        webhook_config: DjangoWebhookConfig = None,
     ):
         if feature_states:
             assert not any(
@@ -240,7 +229,7 @@ def __init__(
         self.api_key = api_key
         self.project = project
         self.feature_states = DjangoFeatureStateRelatedManager(feature_states or [])
-        self.webhooks = DjangoWebhookRelatedManager(webhooks or [])
+        self.webhook_config = webhook_config
 
 
 @dataclass

tests/unit/api/conftest.py

@@ -20,7 +20,7 @@
     DjangoSegmentCondition,
     DjangoSegmentRule,
     DjangoTrait,
-    DjangoWebhook,
+    DjangoWebhookConfig,
 )
 
 
@@ -95,7 +95,7 @@ def django_multivariate_feature_state(django_project):
 
 @pytest.fixture
 def django_webhook():
-    return DjangoWebhook(url="https://my.webhook.com/hook", secret="secret!")
+    return DjangoWebhookConfig(url="https://my.webhook.com/hook", secret="secret!")
 
 
 @pytest.fixture()
@@ -118,7 +118,7 @@ def django_environment(
             django_multivariate_feature_state,
             django_enabled_feature_state_with_string_value,
         ],
-        webhooks=[django_webhook],
+        webhook_config=django_webhook,
     )
 
 

tests/unit/api/test_document_builders.py

@@ -77,11 +77,9 @@ def test_build_environment_document(
             )
         )
 
-    assert len(environment_document["webhooks"]) == 1
-    serialized_webhook = environment_document["webhooks"][0]
-    assert serialized_webhook["enabled"] == django_webhook.enabled
-    assert serialized_webhook["url"] == django_webhook.url
-    assert serialized_webhook["secret"] == django_webhook.secret
+    assert environment_document["webhook_config"] is not None
+    assert environment_document["webhook_config"]["url"] == django_webhook.url
+    assert environment_document["webhook_config"]["secret"] == django_webhook.secret
 
 
 def test_build_environment_api_key_document(django_environment_api_key):

tests/unit/environments/test_environments_builders.py

@@ -14,6 +14,7 @@
 def test_build_environment_model():
     """Test to exercise the basic fields on the schema."""
     # Given
+    webhook_url = "https://my.webhook.com/hook"
     environment_dict = {
         "id": 1,
         "api_key": "api-key",
@@ -37,15 +38,10 @@ def test_build_environment_model():
                 "feature": {"id": 1, "name": "enabled_feature", "type": STANDARD},
             }
         ],
-        "webhooks": [
-            {
-                "url": "https://my.webhook.com/hook",
-                "secret": "secret!",
-                "created_at": "2022-05-11T11:33:14.073487+00:00",
-                "updated_at": "2022-05-11T11:33:14.073487+00:00",
-                "enabled": True,
-            }
-        ],
+        "webhook_config": {
+            "url": webhook_url,
+            "secret": "secret!",
+        },
     }
 
     # When
@@ -55,7 +51,7 @@ def test_build_environment_model():
     assert environment_model
 
     assert len(environment_model.feature_states) == 1
-    assert len(environment_model.webhooks) == 1
+    assert environment_model.webhook_config.url == webhook_url
 
 
 def test_get_flags_for_environment_returns_feature_states_for_environment_dictionary():

tests/unit/segments/test_segments_models.py

@@ -10,12 +10,14 @@
         (constants.EQUAL, "bar", "bar", True),
         (constants.EQUAL, "bar", "baz", False),
         (constants.EQUAL, 1, "1", True),
+        (constants.EQUAL, 1, "not_an_int", False),
         (constants.EQUAL, 1, "2", False),
         (constants.EQUAL, True, "True", True),
         (constants.EQUAL, False, "False", True),
         (constants.EQUAL, False, "True", False),
         (constants.EQUAL, True, "False", False),
         (constants.EQUAL, 1.23, "1.23", True),
+        (constants.EQUAL, 1.23, "not_a_float", False),
         (constants.EQUAL, 1.23, "4.56", False),
         (constants.GREATER_THAN, 2, "1", True),
         (constants.GREATER_THAN, 1, "1", False),
@@ -75,6 +77,7 @@ def test_segment_condition_matches_trait_value(
     "operator, trait_value, condition_value, expected_result",
     [
         (constants.EQUAL, "1.0.0", "1.0.0:semver", True),
+        (constants.EQUAL, "not_a_semver", "1.0.0:semver", False),
         (constants.EQUAL, "1.0.0", "1.0.1:semver", False),
         (constants.NOT_EQUAL, "1.0.0", "1.0.0:semver", False),
         (constants.NOT_EQUAL, "1.0.0", "1.0.1:semver", True),