Skip to content

Commit d365c31

Browse files
janiszclaude
andcommitted
Fix linter issues in test_helpers.go
- Add blank identifier for ignored Close error in cleanup - Split long function signature across multiple lines Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
1 parent 98cb1a3 commit d365c31

File tree

1 file changed

+8
-3
lines changed

1 file changed

+8
-3
lines changed

internal/testutil/test_helpers.go

Lines changed: 8 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -8,19 +8,24 @@ import (
88
"github.com/stretchr/testify/require"
99
)
1010

11-
// SetupInitializedClient creates an initialized MCP client for testing with automatic cleanup.
11+
// SetupInitializedClient creates an initialized MCP client with automatic cleanup.
1212
func SetupInitializedClient(t *testing.T, createClient func(*testing.T) (*MCPTestClient, error)) *MCPTestClient {
1313
t.Helper()
1414

1515
client, err := createClient(t)
1616
require.NoError(t, err, "Failed to create MCP client")
17-
t.Cleanup(func() { client.Close() })
17+
t.Cleanup(func() { _ = client.Close() })
1818

1919
return client
2020
}
2121

2222
// CallToolAndGetResult calls a tool and verifies it succeeds.
23-
func CallToolAndGetResult(t *testing.T, client *MCPTestClient, toolName string, args map[string]any) *mcp.CallToolResult {
23+
func CallToolAndGetResult(
24+
t *testing.T,
25+
client *MCPTestClient,
26+
toolName string,
27+
args map[string]any,
28+
) *mcp.CallToolResult {
2429
t.Helper()
2530

2631
ctx := context.Background()

0 commit comments

Comments
 (0)