Merge pull request #7 from splitio/add-segment-tags

Added tags property support for segment object.

Author: chillaq

splitapiclient/resources/segment.py

@@ -12,9 +12,11 @@ class Segment(BaseResource):
         'description': 'string',
         'trafficType' : {
             'id': 'string',
-            'namr': 'string'
+            'name': 'string'
         },
-        'workspaceId' : 'string'
+        'workspaceId' : 'string',
+        'creationTime' : 'number',
+        'tags': [{'name': 'string'}]
     }
 
     def __init__(self, data=None, client=None):
@@ -27,6 +29,9 @@ def __init__(self, data=None, client=None):
         self._description = data.get('description')
         self._trafficType = TrafficType(data.get('trafficType')) if 'trafficType' in data else {}
         self._workspace_id = data.get('workspaceId')
+        self._tags = data.get('tags') if 'tags' in data else []
+        self._creationTime = data.get('creationTime') if 'creationTime' in data else 0
+
 
     @property
     def name(self):

splitapiclient/tests/microclients/segment_microclient_test.py

@@ -16,9 +16,13 @@ def test_list(self, mocker):
         data = {'objects': [{
                 'name': 'seg1',
                 'description': 'desc',
+                'creationTime' : None,
+                'tags': None
                 }, {
                 'name': 'seg1',
                 'description': 'desc',
+                'creationTime' : None,
+                'tags': None
             }],
             'offset': 1,
             'totalCount': 2,
@@ -36,12 +40,16 @@ def test_list(self, mocker):
                 'name': 'seg1',
                 'description': 'desc',
                 'trafficType': None,
-                'workspaceId': None
+                'workspaceId': None,
+                'creationTime' : None,
+                'tags': None
                 }, {
                 'name': 'seg1',
                 'description': 'desc',
                 'trafficType': None,
-                'workspaceId': None
+                'workspaceId': None,
+                'creationTime' : None,
+                'tags': None
         }]
 
         assert result[0].to_dict() == data[0]

splitapiclient/tests/resources/test_workspace.py

@@ -111,6 +111,8 @@ def test_add_segment(self, mocker):
         data = {
             'name': '1',
             'description': 'e1',
+            'creationTime' : None,
+            'tags': None
         }
         http_client_mock = mocker.Mock(spec=BaseHttpClient)
         http_client_mock.make_request.return_value = data
@@ -133,6 +135,8 @@ def test_add_segment(self, mocker):
         data = {
             'name': '1',
             'description': 'e1',
+            'creationTime' : None,
+            'tags': None
         }
         data['trafficType'] = None
         data['workspaceId'] = None
@@ -198,6 +202,7 @@ def test_add_split(self, mocker):
         data['rolloutStatus'] = None
         data['rolloutStatusTimestamp'] = None
         data['tags'] = None
+        data['owners'] = None
 
         assert attr.to_dict() == data