remove new caching

Author: renaudhartert-db

databricks/sdk/credentials_provider.py

@@ -215,7 +215,6 @@ def external_browser(cfg: 'Config') -> Optional[CredentialsProvider]:
             # to detect if the token is expired and needs to be refreshed
             # by going through the OAuth login flow.
             credentials.token()
-            token_cache.save(credentials)
             return credentials(cfg)
         # TODO: We should ideally use more specific exceptions.
         except Exception as e:

tests/test_credentials_provider.py

@@ -30,7 +30,6 @@ def test_external_browser_refresh_success(mocker):
     got_credentials_provider = external_browser(mock_cfg)
 
     mock_token_cache.load.assert_called_once()
-    mock_token_cache.save.assert_called_once_with(mock_session_credentials)
     mock_session_credentials.token.assert_called_once() # Verify token refresh was attempted
     assert got_credentials_provider == want_credentials_provider