Updated validation and model

Author: chillaq

splitio/client/input_validator.py

@@ -564,7 +564,7 @@ def validate_factory_instantiation(sdk_key):
     return True
 
 
-def valid_properties(properties):
+def valid_properties(properties, source):
     """
     Check if properties is a valid dict and returns the properties
     that will be sent to the track method, avoiding unexpected types.
@@ -580,7 +580,7 @@ def valid_properties(properties):
         return True, None, size
 
     if not isinstance(properties, dict):
-        _LOGGER.error('track: properties must be of type dictionary.')
+        _LOGGER.error('%s: properties must be of type dictionary.', source)
         return False, None, 0
 
     valid_properties = dict()
@@ -597,7 +597,7 @@ def valid_properties(properties):
 
         if not isinstance(element, str) and not isinstance(element, Number) \
            and not isinstance(element, bool):
-            _LOGGER.warning('Property %s is of invalid type. Setting value to None', element)
+            _LOGGER.warning('%s: Property %s is of invalid type. Setting value to None', source, element)
             element = None
 
         valid_properties[property] = element
@@ -607,14 +607,13 @@ def valid_properties(properties):
 
         if size > MAX_PROPERTIES_LENGTH_BYTES:
             _LOGGER.error(
-                'The maximum size allowed for the properties is 32768 bytes. ' +
-                'Current one is ' + str(size) + ' bytes. Event not queued'
-            )
+                '%s: The maximum size allowed for the properties is 32768 bytes. ' +
+                'Current one is ' + str(size) + ' bytes. Event not queued', source)
             return False, None, size
 
     if len(valid_properties.keys()) > 300:
-        _LOGGER.warning('Event has more than 300 properties. Some of them will be trimmed' +
-                        ' when processed')
+        _LOGGER.warning('%s: Event has more than 300 properties. Some of them will be trimmed' +
+                        ' when processed', source)
     return True, valid_properties if len(valid_properties) else None, size
 
 def validate_pluggable_adapter(config):

splitio/models/impressions.py

@@ -12,7 +12,8 @@
         'change_number',
         'bucketing_key',
         'time',
-        'previous_time'
+        'previous_time',
+        'impression_properties'
     ]
 )
 

tests/client/test_input_validator.py

@@ -499,17 +499,17 @@ def _configs(treatment):
 
     def test_valid_properties(self, mocker):
         """Test valid_properties() method."""
-        assert input_validator.valid_properties(None) == (True, None, 1024)
-        assert input_validator.valid_properties([]) == (False, None, 0)
-        assert input_validator.valid_properties(True) == (False, None, 0)
-        assert input_validator.valid_properties(dict()) == (True, None, 1024)
-        assert input_validator.valid_properties({2: 123}) == (True, None, 1024)
+        assert input_validator.valid_properties(None, '') == (True, None, 1024)
+        assert input_validator.valid_properties([], '') == (False, None, 0)
+        assert input_validator.valid_properties(True, '') == (False, None, 0)
+        assert input_validator.valid_properties(dict(), '') == (True, None, 1024)
+        assert input_validator.valid_properties({2: 123}, '') == (True, None, 1024)
 
         class Test:
             pass
         assert input_validator.valid_properties({
             "test": Test()
-        }) == (True, {"test": None}, 1028)
+        }, '') == (True, {"test": None}, 1028)
 
         props1 = {
             "test1": "test",
@@ -519,7 +519,7 @@ class Test:
             "test5": [],
             2: "t",
         }
-        r1, r2, r3 = input_validator.valid_properties(props1)
+        r1, r2, r3 = input_validator.valid_properties(props1, '')
         assert r1 is True
         assert len(r2.keys()) == 5
         assert r2["test1"] == "test"
@@ -532,12 +532,12 @@ class Test:
         props2 = dict()
         for i in range(301):
             props2[str(i)] = i
-        assert input_validator.valid_properties(props2) == (True, props2, 1817)
+        assert input_validator.valid_properties(props2, '') == (True, props2, 1817)
 
         props3 = dict()
         for i in range(100, 210):
             props3["prop" + str(i)] = "a" * 300
-        r1, r2, r3 = input_validator.valid_properties(props3)
+        r1, r2, r3 = input_validator.valid_properties(props3, '')
         assert r1 is False
         assert r3 == 32952