added manager tests

Author: Matias Melograno

tests/client/test_factory.py

@@ -4,7 +4,8 @@
 
 import time
 import threading
-from splitio.client.factory import get_factory, SplitFactory, _INSTANTIATED_FACTORIES, Status
+from splitio.client.factory import get_factory, SplitFactory, _INSTANTIATED_FACTORIES, Status,\
+    _LOGGER as _logger
 from splitio.client.config import DEFAULT_CONFIG
 from splitio.storage import redis, inmemmory, uwsgi
 from splitio.tasks import events_sync, impressions_sync, split_sync, segment_sync, telemetry_sync
@@ -558,3 +559,17 @@ def _recreate(self):
         assert clear_impressions._called == 1
         assert clear_events._called == 1
         assert clear_telemetry._called == 1
+
+    def test_error_prefork(self, mocker):
+        """Test not handling fork."""
+        expected_msg = [
+            mocker.call('Cannot call handle_post_fork')
+        ]
+
+        factory = get_factory("localhost")
+        factory.block_until_ready(1)
+
+        _logger = mocker.Mock()
+        mocker.patch('splitio.client.factory._LOGGER', new=_logger)
+        factory.handle_post_fork()
+        assert _logger.warning.mock_calls == expected_msg

tests/client/test_manager.py

@@ -0,0 +1,36 @@
+"""SDK main manager test module."""
+
+from splitio.client.factory import SplitFactory
+from splitio.client.manager import SplitManager, _LOGGER as _logger
+
+
+class ManagerTests(object):  # pylint: disable=too-few-public-methods
+    """Split manager test cases."""
+
+    def test_evaluations_before_running_post_fork(self, mocker):
+        destroyed_property = mocker.PropertyMock()
+        destroyed_property.return_value = False
+
+        factory = mocker.Mock(spec=SplitFactory)
+        factory._waiting_fork.return_value = True
+        type(factory).destroyed = destroyed_property
+
+        expected_msg = [
+            mocker.call('Client is not ready - no calls possible')
+        ]
+
+        manager = SplitManager(factory)
+        _logger = mocker.Mock()
+        mocker.patch('splitio.client.manager._LOGGER', new=_logger)
+
+        assert manager.split_names() == []
+        assert _logger.error.mock_calls == expected_msg
+        _logger.reset_mock()
+
+        assert manager.split('some_feature') is None
+        assert _logger.error.mock_calls == expected_msg
+        _logger.reset_mock()
+
+        assert manager.splits() == []
+        assert _logger.error.mock_calls == expected_msg
+        _logger.reset_mock()