stacklok / toolhive / 19338685502

51%

Build Type

push

github

Committed by web-flow

Commit Message

Inject env.Reader for testable token exchange auth (#2571)

Refactors environment variable access in token exchange
authentication to use dependency-injected env.Reader
interface instead of direct os.Getenv calls.

Changes:
- TokenExchangeStrategy accepts env.Reader in constructor
- Factory passes env.Reader when creating strategies
- YAMLLoader uses env.Reader for environment variable access
- Commands.go creates OSReader and wires dependencies

This enables proper testing with mocked environment variables
using gomock instead of t.Setenv, allowing parallel test
execution. No functional changes - client_secret_env support
was already present.

Run Details

20 of 29 new or added lines in 4 files covered. (68.97%)

9 existing lines in 3 files now uncovered.

23113 of 48162 relevant lines covered (47.99%)

36.39 hits per line