changed log level to warning when split is not present

Author: Matias Melograno

splitio/client/input_validator.py

@@ -268,7 +268,7 @@ def validate_feature_name(feature_name, should_validate_existance, split_storage
         return None
 
     if should_validate_existance and split_storage.get(feature_name) is None:
-        _LOGGER.error(
+        _LOGGER.warning(
             "%s: you passed \"%s\" that does not exist in this environment, "
             "please double check what Splits exist in the web console.",
             operation,
@@ -380,7 +380,7 @@ def validate_manager_feature_name(feature_name, should_validate_existance, split
         return None
 
     if should_validate_existance and split_storage.get(feature_name) is None:
-        _LOGGER.error(
+        _LOGGER.warning(
             "split: you passed \"%s\" that does not exist in this environment, "
             "please double check what Splits exist in the web console.",
             feature_name
@@ -421,7 +421,7 @@ def validate_features_get_treatments(features, should_validate_existance=False,
 
     valid_missing_features = set(f for f in filtered_features if split_storage.get(f) is None)
     for missing_feature in valid_missing_features:
-        _LOGGER.error(
+        _LOGGER.warning(
             "%s: you passed \"%s\" that does not exist in this environment, "
             "please double check what Splits exist in the web console.",
             operation,

tests/client/test_input_validator.py

@@ -233,7 +233,7 @@ def _get_storage_mock(storage):
         client._logger.reset_mock()
         storage_mock.get.return_value = None
         assert client.get_treatment('matching_key', 'some_feature', None) == CONTROL
-        assert client._logger.error.mock_calls == [
+        assert client._logger.warning.mock_calls == [
             mocker.call(
                 "%s: you passed \"%s\" that does not exist in this environment, "
                 "please double check what Splits exist in the web console.",
@@ -461,7 +461,7 @@ def _get_storage_mock(storage):
         client._logger.reset_mock()
         storage_mock.get.return_value = None
         assert client.get_treatment_with_config('matching_key', 'some_feature', None) == (CONTROL, None)
-        assert client._logger.error.mock_calls == [
+        assert client._logger.warning.mock_calls == [
             mocker.call(
                 "%s: you passed \"%s\" that does not exist in this environment, "
                 "please double check what Splits exist in the web console.",
@@ -788,7 +788,7 @@ def _get_storage_mock(storage):
         ready_mock.return_value = True
         type(factory_mock).ready = ready_mock
         assert client.get_treatments('matching_key', ['some_feature'], None) == {'some_feature': CONTROL}
-        assert client._logger.error.mock_calls == [
+        assert client._logger.warning.mock_calls == [
             mocker.call(
                 "%s: you passed \"%s\" that does not exist in this environment, "
                 "please double check what Splits exist in the web console.",
@@ -909,7 +909,7 @@ def _configs(treatment):
         ready_mock.return_value = True
         type(factory_mock).ready = ready_mock
         assert client.get_treatments('matching_key', ['some_feature'], None) == {'some_feature': CONTROL}
-        assert client._logger.error.mock_calls == [
+        assert client._logger.warning.mock_calls == [
             mocker.call(
                 "%s: you passed \"%s\" that does not exist in this environment, "
                 "please double check what Splits exist in the web console.",
@@ -971,7 +971,7 @@ def test_split_(self, mocker):
         storage_mock.get.return_value = None
         manager.split('nonexistant-split')
         assert split_mock.to_split_view.mock_calls == []
-        assert manager._logger.error.mock_calls == [mocker.call(
+        assert manager._logger.warning.mock_calls == [mocker.call(
             "split: you passed \"%s\" that does not exist in this environment, "
             "please double check what Splits exist in the web console.",
             'nonexistant-split'