stacklok / toolhive / 24566540368

66%

Build Type

push

github

Committed by web-flow

Commit Message

Add unit tests for pkg/vmcp/cli serve and validate logic (#4898)

* Add unit tests for pkg/vmcp/cli serve and validate logic

Cover loadAndValidateConfig, loadAuthServerConfig, discoverBackends,
createSessionFactory, and Validate with table-driven unit tests in
pkg/vmcp/cli. Also moves the existing loadAuthServerConfig test from
cmd/vmcp/app (where it lived before the thin-wrap refactor) to
pkg/vmcp/cli where the function now lives.

env var tests use os.Setenv + t.Cleanup via a setEnvForTest helper
to remain compatible with t.Parallel().

Part of #4808. Closes #4881.

* fixes from review

---------

Co-authored-by: taskbot <taskbot@users.noreply.github.com>

Coverage Stats

10 of 17 new or added lines in 1 file covered. (58.82%)

37 existing lines in 5 files now uncovered.

57665 of 86997 relevant lines covered (66.28%)

60.88 hits per line