cleanup

Author: chillaq

splitio/storage/pluggable.py

@@ -315,7 +315,7 @@ def get_segment_names(self):
 #            _LOGGER.debug('Error: ', exc_info=True)
 #            return None
 
-    def segment_contains_key(self, segment_name, key):
+    def segment_contains(self, segment_name, key):
         """
         Check if segment contains a key
 
@@ -334,20 +334,6 @@ def segment_contains_key(self, segment_name, key):
             _LOGGER.debug('Error: ', exc_info=True)
             return None
 
-    def segment_contains(self, segment_name, key):
-        """
-        Check if segment contains a key, added for backward compatibility as its implemented in Redis
-
-        :param segment_name: segment name
-        :type segment_name: str
-        :param key: key
-        :type key: str
-
-        :return: True if found, otherwise False
-        :rtype: bool
-        """
-        self.segment_contains_key(segment_name, key)
-
     def get_segment_keys_count(self):
         """
         Get count of all keys in segments.

tests/storage/test_pluggable.py

@@ -295,7 +295,6 @@ def test_get_segment_names(self):
     def test_segment_contains(self):
         self.mock_adapter._keys = {}
         self.pluggable_segment_storage.update('segment1', ['key1', 'key2'], [], 123)
-        assert(self.pluggable_segment_storage.segment_contains_key('segment1', 'key1'))
         assert(not self.pluggable_segment_storage.segment_contains('segment1', 'key5'))
 
     # TODO: To be added when producer mode is implemented