opencode/picoclaw
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

fix(mcp): close response body in headerTransport tests

Browse source 
Fixes bodyclose linter warnings.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
This commit is contained in:
Andy Lo-A-Foe 2026-04-28 10:17:21 +02:00
parent 07422a5403
commit a66fbbef0c
No known key found for this signature in database
GPG key ID: C0E4EB79E9E6A23D
1 changed files with 6 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

9
pkg/mcp/manager_test.go
View file

@ -651,10 +651,11 @@ func TestHeaderTransport_DynamicHeaders(t *testing.T) {
}) })
req, _ := http.NewRequestWithContext(ctx, http.MethodPost, "http://example.com", nil) req, _ := http.NewRequestWithContext(ctx, http.MethodPost, "http://example.com", nil)
_, err := transport.RoundTrip(req) resp, err := transport.RoundTrip(req)
if err != nil { if err != nil {
t.Fatalf("RoundTrip() error = %v", err) t.Fatalf("RoundTrip() error = %v", err)
} }
resp.Body.Close()
if got := captured.Get("X-Static"); got != "from-config" { if got := captured.Get("X-Static"); got != "from-config" {
t.Errorf("X-Static = %q, want %q", got, "from-config") t.Errorf("X-Static = %q, want %q", got, "from-config")
@ -686,10 +687,11 @@ func TestHeaderTransport_DynamicOverridesStatic(t *testing.T) {
}) })
req, _ := http.NewRequestWithContext(ctx, http.MethodPost, "http://example.com", nil) req, _ := http.NewRequestWithContext(ctx, http.MethodPost, "http://example.com", nil)
_, err := transport.RoundTrip(req) resp, err := transport.RoundTrip(req)
if err != nil { if err != nil {
t.Fatalf("RoundTrip() error = %v", err) t.Fatalf("RoundTrip() error = %v", err)
} }
resp.Body.Close()
if got := captured.Get("Authorization"); got != "Bearer dynamic" { if got := captured.Get("Authorization"); got != "Bearer dynamic" {
t.Errorf("Authorization = %q, want dynamic to override static %q", got, "Bearer dynamic") t.Errorf("Authorization = %q, want dynamic to override static %q", got, "Bearer dynamic")
@ -711,10 +713,11 @@ func TestHeaderTransport_NoDynamicHeaders(t *testing.T) {
} }
req, _ := http.NewRequestWithContext(context.Background(), http.MethodPost, "http://example.com", nil) req, _ := http.NewRequestWithContext(context.Background(), http.MethodPost, "http://example.com", nil)
_, err := transport.RoundTrip(req) resp, err := transport.RoundTrip(req)
if err != nil { if err != nil {
t.Fatalf("RoundTrip() error = %v", err) t.Fatalf("RoundTrip() error = %v", err)
} }
resp.Body.Close()
if got := captured.Get("X-Static"); got != "val" { if got := captured.Get("X-Static"); got != "val" {
t.Errorf("X-Static = %q, want %q", got, "val") t.Errorf("X-Static = %q, want %q", got, "val")