Merge pull request #252 from splitio/developmentRC

Development rc

Author: mmelograno

splitio/api/auth.py

@@ -40,7 +40,7 @@ def authenticate(self):
         try:
             response = self._client.get(
                 'auth',
-                '/auth',
+                '/v2/auth',
                 self._apikey,
                 extra_headers=self._metadata
             )

splitio/client/factory.py

@@ -390,7 +390,7 @@ def _build_redis_factory(api_key, cfg):
     }
     data_sampling = cfg.get('dataSampling', DEFAULT_DATA_SAMPLING)
     if data_sampling < _MIN_DEFAULT_DATA_SAMPLING_ALLOWED:
-        _LOGGER.warning("dataSampling cannot be less than %f, defaulting to minimum",
+        _LOGGER.warning("dataSampling cannot be less than %.2f, defaulting to minimum",
                         _MIN_DEFAULT_DATA_SAMPLING_ALLOWED)
         data_sampling = _MIN_DEFAULT_DATA_SAMPLING_ALLOWED
     recorder = PipelinedRecorder(

tests/api/test_auth.py

@@ -29,7 +29,7 @@ def test_auth(self, mocker):
         call_made = httpclient.get.mock_calls[0]
 
         # validate positional arguments
-        assert call_made[1] == ('auth', '/auth', 'some_api_key')
+        assert call_made[1] == ('auth', '/v2/auth', 'some_api_key')
 
         # validate key-value args (headers)
         assert call_made[2]['extra_headers'] == {

tests/helpers/mockserver.py

@@ -220,7 +220,7 @@ def do_GET(self):  #pylint:disable=invalid-name
             self._handle_split_changes()
         elif self.path.startswith('/api/segmentChanges'):
             self._handle_segment_changes()
-        elif self.path.startswith('/api/auth'):
+        elif self.path.startswith('/api/v2/auth'):
             self._handle_auth()
         else:
             self.send_response(404)

tests/integration/test_streaming_e2e.py

@@ -148,7 +148,7 @@ def test_happiness(self):
         # Auth
         req = split_backend_requests.get()
         assert req.method == 'GET'
-        assert req.path == '/api/auth'
+        assert req.path == '/api/v2/auth'
         assert req.headers['authorization'] == 'Bearer some_apikey'
 
         # SyncAll after streaming connected
@@ -355,7 +355,7 @@ def test_occupancy_flicker(self):
         # Auth
         req = split_backend_requests.get()
         assert req.method == 'GET'
-        assert req.path == '/api/auth'
+        assert req.path == '/api/v2/auth'
         assert req.headers['authorization'] == 'Bearer some_apikey'
 
         # SyncAll after streaming connected
@@ -535,7 +535,7 @@ def test_start_without_occupancy(self):
         # Auth
         req = split_backend_requests.get()
         assert req.method == 'GET'
-        assert req.path == '/api/auth'
+        assert req.path == '/api/v2/auth'
         assert req.headers['authorization'] == 'Bearer some_apikey'
 
         # SyncAll after streaming connected
@@ -725,7 +725,7 @@ def test_streaming_status_changes(self):
         # Auth
         req = split_backend_requests.get()
         assert req.method == 'GET'
-        assert req.path == '/api/auth'
+        assert req.path == '/api/v2/auth'
         assert req.headers['authorization'] == 'Bearer some_apikey'
 
         # SyncAll after streaming connected
@@ -952,7 +952,7 @@ def test_server_closes_connection(self):
         # Auth
         req = split_backend_requests.get()
         assert req.method == 'GET'
-        assert req.path == '/api/auth'
+        assert req.path == '/api/v2/auth'
         assert req.headers['authorization'] == 'Bearer some_apikey'
 
         # SyncAll after streaming connected
@@ -982,7 +982,7 @@ def test_server_closes_connection(self):
         # Auth after connection breaks
         req = split_backend_requests.get()
         assert req.method == 'GET'
-        assert req.path == '/api/auth'
+        assert req.path == '/api/v2/auth'
         assert req.headers['authorization'] == 'Bearer some_apikey'
 
         # SyncAll after streaming connected again
@@ -1189,7 +1189,7 @@ def test_ably_errors_handling(self):
         # Auth
         req = split_backend_requests.get()
         assert req.method == 'GET'
-        assert req.path == '/api/auth'
+        assert req.path == '/api/v2/auth'
         assert req.headers['authorization'] == 'Bearer some_apikey'
 
         # SyncAll after streaming connected
@@ -1213,7 +1213,7 @@ def test_ably_errors_handling(self):
         # Auth again
         req = split_backend_requests.get()
         assert req.method == 'GET'
-        assert req.path == '/api/auth'
+        assert req.path == '/api/v2/auth'
         assert req.headers['authorization'] == 'Bearer some_apikey'
 
         # SyncAll after push is up