stacklok / toolhive / 28030733319
67%

DEFAULT BRANCH: main
Ran
Jobs 1
Files 772
Run time 2min
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: 67.285% (+0.02%) from 67.261%
28030733319

push

github

web-flow 
Add reconciler test helpers to cut fake-client boilerplate (#5584)

* Add fake-client reconciler test helpers

Controller tests hand-rolled an 8-10 line fake-client + reconciler
assembly block at every call site. Add newTestVirtualMCPServerReconciler
and newTestMCPRemoteProxyReconciler, which build the fake client
internally and return both the reconciler and the client, collapsing each
block to one line. Migrate the matching call sites.

Only blocks whose fake client registers exactly the reconciler's own type
as a status subresource are migrated; blocks with extra status
subresources, custom interceptors, or indexes keep their inline builders.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

* Cover config controllers with reconciler test helpers

Rename mcpserver_test_helpers_test.go to reconciler_test_helpers_test.go
to reflect that it serves every controller test, not just MCPServer, and
factor the shared fake-client build into newTestFakeClient.

Add factories for the config-tier reconcilers (MCPExternalAuthConfig,
MCPOIDCConfig, MCPAuthzConfig), which carry Client/Scheme/Recorder rather
than the workload tier's PlatformDetector. Recorder is left nil so callers
that assert on events set it explicitly. Migrate the matching call sites.

As before, only blocks registering exactly the reconciler's own type as a
status subresource are migrated; absent, multi-type, or interceptor-based
builders keep their inline construction.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

---------

Co-authored-by: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

70142 of 104246 relevant lines covered (67.29%)

65.61 hits per line

Coverage Regressions

Lines Coverage ∆ File
3
80.42
 -0.71% pkg/transport/proxy/httpsse/http_proxy.go
2
93.94
 -6.06% pkg/foreach/foreach.go
Jobs
ID Job ID Ran Files Coverage
1 28030733319.1 772
67.29
 GitHub Action Run
Source Files on build 28030733319