Skip to content

Commit 390f498

Browse files
committed
replaced split with feature flag
1 parent 000ac80 commit 390f498

File tree

2 files changed

+19
-19
lines changed

2 files changed

+19
-19
lines changed

splitio/push/parser.py

Lines changed: 16 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -276,7 +276,7 @@ def __str__(self):
276276

277277

278278
class BaseUpdate(BaseMessage, metaclass=abc.ABCMeta):
279-
"""Split data update notification."""
279+
"""Feature flag data update notification."""
280280

281281
def __init__(self, channel, timestamp, change_number):
282282
"""
@@ -323,13 +323,13 @@ def change_number(self):
323323

324324

325325
class SplitChangeUpdate(BaseUpdate):
326-
"""Split Change notification."""
326+
"""Feature flag Change notification."""
327327

328-
def __init__(self, channel, timestamp, change_number, previous_change_number, split_definition, compression):
328+
def __init__(self, channel, timestamp, change_number, previous_change_number, feature_flag_definition, compression):
329329
"""Class constructor."""
330330
BaseUpdate.__init__(self, channel, timestamp, change_number)
331331
self._previous_change_number = previous_change_number
332-
self._split_definition = split_definition
332+
self._feature_flag_definition = feature_flag_definition
333333
self._compression = compression
334334

335335
@property
@@ -353,14 +353,14 @@ def previous_change_number(self): # pylint:disable=no-self-use
353353
return self._previous_change_number
354354

355355
@property
356-
def split_definition(self): # pylint:disable=no-self-use
356+
def feature_flag_definition(self): # pylint:disable=no-self-use
357357
"""
358-
Return split definition
358+
Return feature flag definition
359359
360-
:returns: The new split definition
360+
:returns: The new feature flag definition
361361
:rtype: str
362362
"""
363-
return self._split_definition
363+
return self._feature_flag_definition
364364

365365
@property
366366
def compression(self): # pylint:disable=no-self-use
@@ -378,12 +378,12 @@ def __str__(self):
378378

379379

380380
class SplitKillUpdate(BaseUpdate):
381-
"""Split Kill notification."""
381+
"""Feature flag Kill notification."""
382382

383-
def __init__(self, channel, timestamp, change_number, split_name, default_treatment): # pylint:disable=too-many-arguments
383+
def __init__(self, channel, timestamp, change_number, feature_flag_name, default_treatment): # pylint:disable=too-many-arguments
384384
"""Class constructor."""
385385
BaseUpdate.__init__(self, channel, timestamp, change_number)
386-
self._split_name = split_name
386+
self._feature_flag_name = feature_flag_name
387387
self._default_treatment = default_treatment
388388

389389
@property
@@ -397,14 +397,14 @@ def update_type(self): # pylint:disable=no-self-use
397397
return UpdateType.SPLIT_KILL
398398

399399
@property
400-
def split_name(self):
400+
def feature_flag_name(self):
401401
"""
402-
Return the name of the killed split.
402+
Return the name of the killed feature flag.
403403
404-
:returns: name of the killed split
404+
:returns: name of the killed feature flag
405405
:rtype: str
406406
"""
407-
return self._split_name
407+
return self._feature_flag_name
408408

409409
@property
410410
def default_treatment(self):
@@ -419,7 +419,7 @@ def default_treatment(self):
419419
def __str__(self):
420420
"""Return string representation."""
421421
return "SplitKill - changeNumber=%d, name=%s, defaultTreatment=%s" % \
422-
(self.change_number, self.split_name, self.default_treatment)
422+
(self.change_number, self.feature_flag, self.default_treatment)
423423

424424

425425
class SegmentChangeUpdate(BaseUpdate):

tests/push/test_parser.py

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -55,7 +55,7 @@ def test_event_parsing(self):
5555
assert isinstance(parsed0, SplitKillUpdate)
5656
assert parsed0.default_treatment == 'some'
5757
assert parsed0.change_number == 1591996754396
58-
assert parsed0.split_name == 'test'
58+
assert parsed0.feature_flag_name == 'test'
5959

6060
e1 = make_message(
6161
'NDA5ODc2MTAyNg==_MzAyODY0NDkyOA==_splits',
@@ -66,7 +66,7 @@ def test_event_parsing(self):
6666
assert parsed1.change_number == 1591996685190
6767
assert parsed1.previous_change_number == 12
6868
assert parsed1.compression == 2
69-
assert parsed1.split_definition == 'eJzEUtFu2kAQ/BU0z4d0hw2Be0MFRVGJIx'
69+
assert parsed1.feature_flag_definition == 'eJzEUtFu2kAQ/BU0z4d0hw2Be0MFRVGJIx'
7070

7171
e1 = make_message(
7272
'NDA5ODc2MTAyNg==_MzAyODY0NDkyOA==_splits',
@@ -77,7 +77,7 @@ def test_event_parsing(self):
7777
assert parsed1.change_number == 1591996685190
7878
assert parsed1.previous_change_number == None
7979
assert parsed1.compression == None
80-
assert parsed1.split_definition == None
80+
assert parsed1.feature_flag_definition == None
8181

8282

8383
e2 = make_message(

0 commit comments

Comments
 (0)