stacklok / toolhive / 19136383232

51%

Build Type

push

github

Committed by web-flow

Commit Message

Refactor auth resolution logic to config struct (#2478)

Move authentication resolution from CLI backend discoverer to
OutgoingAuthConfig.ResolveForBackend() method for better separation
of concerns and reusability.

The resolveAuthConfig method in cli_discoverer.go contained logic
that belongs in the config struct. This refactoring allows the auth
resolution logic to be reused by any component that needs to resolve
backend authentication, not just the CLI discoverer.

Changes:
- Add ResolveForBackend() method to OutgoingAuthConfig
- Update CLI discoverer to use new method
- Add comprehensive test coverage for auth resolution logic
- Remove duplicate resolveAuthConfig() method from cli_discoverer

Fixes #2465

Run Details

15 of 16 new or added lines in 2 files covered. (93.75%)

8 existing lines in 3 files now uncovered.

22117 of 45680 relevant lines covered (48.42%)

37.12 hits per line