Fixed tests

Author: chillaq

splitio/engine/impressions/strategies.py

@@ -100,5 +100,5 @@ def process_impressions(self, impressions):
         """
         imps = [(self._observer.test_and_set(imp), attrs) for imp, attrs in impressions]
         self._counter.track([imp for imp, _ in imps if imp.previous_time != None])
-        this_hour = truncate_time(util.utctime_ms())
+        this_hour = truncate_time(utctime_ms())
         return [i for i, _ in imps if i.previous_time is None or i.previous_time < this_hour], imps

tests/client/test_input_validator.py

@@ -1179,7 +1179,8 @@ def test_input_validation_factory(self, mocker):
 
         logger.reset_mock()
         try:
-            f = get_factory(True, config={'redisHost': 'some-host'})
+            f = get_factory(True, config={'redisHost': 'localhost'})
         except:
             pass
         assert logger.error.mock_calls == []
+        f.destroy()

tests/integration/test_client_e2e.py

@@ -92,7 +92,10 @@ def _validate_last_impressions(self, client, *to_validate):
 
     def test_get_treatment(self):
         """Test client.get_treatment()."""
-        client = self.factory.client()
+        try:
+            client = self.factory.client()
+        except:
+            pass
 
         assert client.get_treatment('user1', 'sample_feature') == 'on'
         self._validate_last_impressions(client, ('sample_feature', 'user1', 'on'))
@@ -135,8 +138,10 @@ def test_get_treatment(self):
 
     def test_get_treatment_with_config(self):
         """Test client.get_treatment_with_config()."""
-        client = self.factory.client()
-
+        try:
+            client = self.factory.client()
+        except:
+            pass
         result = client.get_treatment_with_config('user1', 'sample_feature')
         assert result == ('on', '{"size":15,"test":20}')
         self._validate_last_impressions(client, ('sample_feature', 'user1', 'on'))
@@ -161,8 +166,10 @@ def test_get_treatment_with_config(self):
 
     def test_get_treatments(self):
         """Test client.get_treatments()."""
-        client = self.factory.client()
-
+        try:
+            client = self.factory.client()
+        except:
+            pass
         result = client.get_treatments('user1', ['sample_feature'])
         assert len(result) == 1
         assert result['sample_feature'] == 'on'
@@ -211,7 +218,10 @@ def test_get_treatments(self):
 
     def test_get_treatments_with_config(self):
         """Test client.get_treatments_with_config()."""
-        client = self.factory.client()
+        try:
+            client = self.factory.client()
+        except:
+            pass
 
         result = client.get_treatments_with_config('user1', ['sample_feature'])
         assert len(result) == 1
@@ -757,7 +767,10 @@ def test_get_treatments_with_config(self):
 
     def test_manager_methods(self):
         """Test manager.split/splits."""
-        manager = self.factory.manager()
+        try:
+            manager = self.factory.manager()
+        except:
+            pass
         result = manager.split('all_feature')
         assert result.name == 'all_feature'
         assert result.traffic_type is None