stacklok / toolhive / 22688788227 / 1
64%
main: 64%

DEFAULT BRANCH: main
Ran
Files 532
Run time 11s
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst
coverage: 63.818% (+0.001%) from 63.817%
22688788227.1

push

github

web-flow 
Fix MCP server tests and remove macos-latest from unit test matrix (#4002)

* Fix MCP server tests to not require container runtime

The MCP server unit tests called New() directly, which internally calls
workloads.NewManager() to detect a container runtime. This caused tests
to fail on macOS GitHub Actions runners where Docker is not pre-installed
(unlike ubuntu-latest runners which have Docker by default).

Extract newServerWithHandler from New() to allow injecting a pre-built
handler. Tests now use mock dependencies via a newTestServer helper,
making them portable across all environments without relying on ambient
infrastructure.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

* Use real runtime on non-darwin in MCP server tests

On macOS (darwin) a container runtime may not be present, so tests use
mock dependencies. On Linux (and other platforms) the real New()
constructor is used, exercising the full stack with an actual runtime.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

* Use real runtime on non-darwin in MCP server tests

On macOS (darwin) a container runtime may not be present, so tests use
mock dependencies. On Linux (and other platforms) the real New()
constructor is used, exercising the full stack with an actual runtime.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

* Use real runtime on non-darwin in MCP server tests

On macOS (darwin) a container runtime may not be present, so tests use
mock dependencies. On Linux (and other platforms) the real New()
constructor is used, exercising the full stack with an actual runtime.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

* Use real runtime on non-darwin in MCP server tests

On macOS (darwin) a container runtime may not be present, so tests use
mock dependencies. On Linux (and other platforms) the real New()
constructor is used, exercising the full stack with an actual runtime.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

... (continued)

46725 of 73216 relevant lines covered (63.82%)

76.04 hits per line

Source Files on job 22688788227.1