Adding coverage to tools package

Author: saucow

CONTRIBUTING.md

@@ -23,6 +23,26 @@ make format
 make lint
 ```
 
+## Testing
+
+### Running Tests
+
+```bash
+# Run all unit tests
+make test
+
+# Run integration tests (requires Docker plugin to be installed)
+make integration
+```
+
+### Unit Test Coverage
+
+```bash
+# Generate HTML coverage report for ALL packages in one view
+go test -cover -coverprofile=coverage.out ./... -short
+go tool cover -html=coverage.out -o coverage.html && open coverage.html
+```
+
 ## Open a Pull Request
 
 1. Fork the repository

cmd/docker-mcp/tools/call_test.go

@@ -0,0 +1,40 @@
+package tools
+
+import (
+	"context"
+	"testing"
+
+	"github.com/modelcontextprotocol/go-sdk/mcp"
+	"github.com/stretchr/testify/assert"
+	"github.com/stretchr/testify/require"
+)
+
+func TestCallNoToolName(t *testing.T) {
+	err := Call(context.Background(), "2", []string{}, false, []string{})
+	require.Error(t, err)
+	assert.Equal(t, "no tool name provided", err.Error())
+}
+
+func TestToText(t *testing.T) {
+	// Test basic functionality - joining multiple text contents
+	response := &mcp.CallToolResult{
+		Content: []mcp.Content{
+			&mcp.TextContent{Text: "First"},
+			&mcp.TextContent{Text: "Second"},
+		},
+	}
+	result := toText(response)
+	assert.Equal(t, "First\nSecond", result)
+}
+
+func TestParseArgs(t *testing.T) {
+	// Test key=value parsing
+	result := parseArgs([]string{"key1=value1", "key2=value2"})
+	expected := map[string]any{"key1": "value1", "key2": "value2"}
+	assert.Equal(t, expected, result)
+
+	// Test duplicate keys become arrays
+	result = parseArgs([]string{"tag=red", "tag=blue"})
+	expected = map[string]any{"tag": []any{"red", "blue"}}
+	assert.Equal(t, expected, result)
+}

cmd/docker-mcp/tools/tools_test.go renamed to cmd/docker-mcp/tools/enable_test.go

@@ -0,0 +1,28 @@
+package tools
+
+import (
+	"testing"
+
+	"github.com/modelcontextprotocol/go-sdk/mcp"
+	"github.com/stretchr/testify/assert"
+)
+
+func TestToolDescription(t *testing.T) {
+	// Test that title annotation takes precedence over description
+	tool := &mcp.Tool{
+		Description: "Longer description",
+		Annotations: &mcp.ToolAnnotations{Title: "Short Title"},
+	}
+	result := toolDescription(tool)
+	assert.Equal(t, "Short Title", result)
+}
+
+func TestDescriptionSummary(t *testing.T) {
+	// Test key behavior: stops at first sentence
+	result := descriptionSummary("First sentence. Second sentence.")
+	assert.Equal(t, "First sentence.", result)
+
+	// Test key behavior: stops at "Error Responses:"
+	result = descriptionSummary("Tool description.\nError Responses:\n- 404 if not found")
+	assert.Equal(t, "Tool description.", result)
+}