fix(databricks): lazy-load the “workbench” profile

[tdstein]

Defer loading of the default “workbench” credentials until they’re actually needed in WorkbenchStrategy.__call__. Previously we loaded the profile eagerly, which could raise if you never used it in a given context.

Resolves #413

[github-actions]

☂️ Python Coverage

current status: ✅

Overall Coverage

Lines	Covered	Coverage	Threshold	Status
1903	1794	94%	0%	🟢

New Files

No new covered files...

Modified Files

File	Coverage	Status
src/posit/workbench/external/databricks.py	86%	🟢
TOTAL	86%	🟢

updated for commit: 7f099e7 by action🐍

[Copilot]

Pull Request Overview

This PR defers loading of the default “workbench” profile until the strategy is invoked and updates tests accordingly.

• Lazy-load Config(profile="workbench") in WorkbenchStrategy._config instead of on init
• Simplify the credentials provider to a lambda in __call__
• Adjust and add tests for default and provided-Config behaviors

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File	Description
tests/posit/workbench/external/test_databricks.py	Rename and expand tests to cover lazy-loading and provided Config
src/posit/workbench/external/databricks.py	Delay instantiation of default Config and return a lambda for credentials

Comments suppressed due to low confidence (3)

tests/posit/workbench/external/test_databricks.py:21

• The test is matching on 'profile=workbench', but the raised error is "Missing value for field 'token' in Config.". Update the regex to match the actual message, e.g., match="Missing value for field 'token'".

with pytest.raises(ValueError, match="profile=workbench"):

tests/posit/workbench/external/test_databricks.py:24

• [nitpick] Test name is ambiguous. Rename it to reflect that it verifies behavior when a Config is provided, e.g., test_with_provided_config_strategy.

def test_workbench_strategy(self):

tests/posit/workbench/external/test_databricks.py:26

• Add an assertion to verify that calling the strategy returns the expected Authorization header, e.g., headers = cs(); assert headers() == {'Authorization': 'Bearer token'}.

cs = WorkbenchStrategy(
    config=Config(host="https://databricks.com/workspace", token="token")