Apply attribute transformer when reading in attributes from the environment (#293)

## Changes
Attributes read from the environment are not currently parsed before
being stored in the `_inner` field of `Config`. As a result, the type of
the attribute will be different depending on whether the attribute is
read from the environment or passed to the constructor. This PR changes
all argument parsing to use `__setattr__` which converts the argument to
the expected type.

Closes #292.

## Tests
Unit test to ensure that string env vars are parsed to appropriate type
for the affected field.

- [ ] `make test` run locally
- [ ] `make fmt` applied
- [ ] relevant integration tests applied

Author: mgyucht

databricks/sdk/core.py

@@ -754,20 +754,19 @@ def _set_inner_config(self, keyword_args: Dict[str, any]):
                 continue
             if keyword_args.get(attr.name, None) is None:
                 continue
-            # make sure that args are of correct type
-            self._inner[attr.name] = attr.transform(keyword_args[attr.name])
+            self.__setattr__(attr.name, keyword_args[attr.name])
 
     def _load_from_env(self):
         found = False
-        for attr in Config.attributes():
+        for attr in self.attributes():
             if not attr.env:
                 continue
             if attr.name in self._inner:
                 continue
             value = os.environ.get(attr.env)
             if not value:
                 continue
-            self._inner[attr.name] = value
+            self.__setattr__(attr.name, value)
             found = True
         if found:
             logger.debug('Loaded from environment')

tests/test_core.py

@@ -11,6 +11,8 @@
                                  databricks_cli)
 from databricks.sdk.version import __version__
 
+from .conftest import noop_credentials
+
 
 def test_parse_dsn():
     cfg = Config.parse_dsn('databricks://user:[email protected]?retry_timeout_seconds=600')
@@ -220,3 +222,9 @@ def __init__(self):
 
     with pytest.raises(ValueError): # As opposed to `KeyError`.
         DatabricksConfig()
+
+
+def test_config_parsing_non_string_env_vars(monkeypatch):
+    monkeypatch.setenv('DATABRICKS_DEBUG_TRUNCATE_BYTES', '100')
+    c = Config(host='http://localhost', credentials_provider=noop_credentials)
+    assert c.debug_truncate_bytes == 100