diff --git a/tests/test_event_builder.py b/tests/test_event_builder.py index ae731df2..00a16f7c 100644 --- a/tests/test_event_builder.py +++ b/tests/test_event_builder.py @@ -404,20 +404,17 @@ def test_create_conversion_event__with_event_value(self): 'name': 'revenue', 'value': 4200 }], - 'eventFeatures': [ - { + 'eventFeatures': [{ 'id': 'non-revenue', 'type': 'custom', 'value': 'abc', 'shouldIndex': False, - }, - { + }, { 'id': 'revenue', 'type': 'custom', 'value': 4200, 'shouldIndex': False, - } - ], + }], 'layerStates': [{ 'layerId': '111182', 'revision': '42', @@ -463,23 +460,22 @@ def test_create_conversion_event__with_invalid_event_value(self): 'visitorId': 'test_user', 'eventName': 'test_event', 'eventEntityId': '111095', + 'revision': '42', 'eventMetrics': [], - 'eventFeatures': [ - { + 'eventFeatures': [{ 'id': 'non-revenue', 'type': 'custom', 'value': 'abc', 'shouldIndex': False, - }, - { + }, { 'id': 'revenue', 'type': 'custom', 'value': '4200', 'shouldIndex': False, - } - ], + }], 'layerStates': [{ 'layerId': '111182', + 'revision': '42', 'decision': { 'experimentId': '111127', 'variationId': '111129', diff --git a/tests/test_optimizely.py b/tests/test_optimizely.py index b89f1ac7..ffd0828b 100644 --- a/tests/test_optimizely.py +++ b/tests/test_optimizely.py @@ -762,6 +762,7 @@ def test_track__with_event_value(self): 'visitorId': 'test_user', 'clientVersion': version.__version__, 'clientEngine': 'python-sdk', + 'revision': '42', 'userFeatures': [{ 'shouldIndex': True, 'type': 'custom', @@ -773,26 +774,24 @@ def test_track__with_event_value(self): 'isGlobalHoldback': False, 'eventEntityId': '111095', 'eventName': 'test_event', - 'eventFeatures': [ - { + 'eventFeatures': [{ 'id': 'non-revenue', 'type': 'custom', 'value': 'abc', 'shouldIndex': False, - }, - { + }, { 'id': 'revenue', 'type': 'custom', 'value': 4200, 'shouldIndex': False, - }, - ], + }], 'eventMetrics': [{ 'name': 'revenue', 'value': 4200 }], 'timestamp': 42000, 'layerStates': [{ + 'revision': '42', 'decision': { 'variationId': '111128', 'isLayerHoldback': False, @@ -834,14 +833,12 @@ def test_track__with_deprecated_event_value(self): 'isGlobalHoldback': False, 'eventEntityId': '111095', 'eventName': 'test_event', - 'eventFeatures': [ - { + 'eventFeatures': [{ 'id': 'revenue', 'type': 'custom', 'value': 4200, 'shouldIndex': False, - } - ], + }], 'eventMetrics': [{ 'name': 'revenue', 'value': 4200 @@ -881,6 +878,7 @@ def test_track__with_invalid_event_value(self): 'visitorId': 'test_user', 'clientVersion': version.__version__, 'clientEngine': 'python-sdk', + 'revision': '42', 'userFeatures': [{ 'shouldIndex': True, 'type': 'custom', @@ -892,17 +890,16 @@ def test_track__with_invalid_event_value(self): 'isGlobalHoldback': False, 'eventEntityId': '111095', 'eventName': 'test_event', - 'eventFeatures': [ - { + 'eventFeatures': [{ 'id': 'revenue', 'type': 'custom', 'value': '4200', 'shouldIndex': False, - } - ], + }], 'eventMetrics': [], 'timestamp': 42000, 'layerStates': [{ + 'revision': '42', 'decision': { 'variationId': '111128', 'isLayerHoldback': False,