1
51%
main: 51%

Ran 12 Nov 2025 08:52PM UTC

Files 380

Run time 13s

Badge

Embed ▾

Committed 12 Nov 2025 08:46PM UTC coverage: 48.921% (-0.001%) from 48.922%

Job # 19311403055.1

Build Type

push

github

Committed by

web-flow

Commit Message

Fix flaky TestGenerateUserAgent test using dependency injection (#2561)

The test was flaky due to race conditions from parallel subtests manipulating
the shared global environment variable KUBERNETES_SERVICE_HOST. This caused
intermittent failures when tests executed concurrently.

This fix refactors generateUserAgent() to use dependency injection with the
existing env.Reader interface, allowing tests to use isolated mock environment
readers instead of modifying global state. The tests now run reliably in
parallel without interference.

Changes:
- Add generateUserAgentWithEnv() that accepts an env.Reader parameter
- Keep generateUserAgent() as a wrapper using env.OSReader for production
- Update tests to use gomock-based environment readers
- Remove all os.Setenv/Unsetenv calls from tests
- Tests remain parallel and pass consistently with race detector

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-authored-by: Claude <noreply@anthropic.com>

Run Details

24163 of 49392 relevant lines covered (48.92%)

35.71 hits per line

stacklok / toolhive / 19311403055 / 1
51%
main: 51%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 19311403055.1

stacklok / toolhive / 19311403055 / 1 51% main: 51%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 19311403055.1

stacklok / toolhive / 19311403055 / 1
51%
main: 51%

README BADGES
x