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>

Author: janisz

internal/testutil/test_helpers.go

@@ -8,19 +8,22 @@ import (
 	"github.com/stretchr/testify/require"
 )
 
-// SetupInitializedClient creates an initialized MCP client for testing with automatic cleanup.
 func SetupInitializedClient(t *testing.T, createClient func(*testing.T) (*MCPTestClient, error)) *MCPTestClient {
 	t.Helper()
 
 	client, err := createClient(t)
 	require.NoError(t, err, "Failed to create MCP client")
-	t.Cleanup(func() { client.Close() })
+	t.Cleanup(func() { _ = client.Close() })
 
 	return client
 }
 
-// CallToolAndGetResult calls a tool and verifies it succeeds.
-func CallToolAndGetResult(t *testing.T, client *MCPTestClient, toolName string, args map[string]any) *mcp.CallToolResult {
+func CallToolAndGetResult(
+	t *testing.T,
+	client *MCPTestClient,
+	toolName string,
+	args map[string]any,
+) *mcp.CallToolResult {
 	t.Helper()
 
 	ctx := context.Background()
@@ -31,7 +34,6 @@ func CallToolAndGetResult(t *testing.T, client *MCPTestClient, toolName string,
 	return result
 }
 
-// GetTextContent extracts text from the first content item.
 func GetTextContent(t *testing.T, result *mcp.CallToolResult) string {
 	t.Helper()
 	require.NotEmpty(t, result.Content, "should have content in response")