Skip to content

Commit 9791244

Browse files
committed
Clean-up test structure update contributing
1 parent 36776cf commit 9791244

File tree

2 files changed

+4
-6
lines changed

2 files changed

+4
-6
lines changed

CONTRIBUTING.md

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -31,12 +31,8 @@ make lint
3131
# Run all unit tests
3232
make test
3333

34-
# Run integration tests (requires Docker plugin to be installed)
34+
# Run integration tests
3535
make integration
36-
37-
# Run integration tests with console output
38-
docker buildx build --build-arg GO_LDFLAGS --build-arg DOCKER_MCP_PLUGIN_BINARY --target=test --progress=plain --no-cache .
39-
4036
```
4137

4238
### Unit Test Coverage

cmd/docker-mcp/tools/tools_test.go

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -320,7 +320,7 @@ func withSampleCatalog() option {
320320
}
321321
}
322322

323-
// Unit tests for utility functions
323+
// Unit tests for call
324324

325325
func TestCallNoToolName(t *testing.T) {
326326
err := Call(context.Background(), "2", []string{}, false, []string{})
@@ -352,6 +352,8 @@ func TestParseArgs(t *testing.T) {
352352
assert.Equal(t, expected, result)
353353
}
354354

355+
// Unit tests for list
356+
355357
func TestToolDescription(t *testing.T) {
356358
// Test that title annotation takes precedence over description
357359
tool := &mcp.Tool{

0 commit comments

Comments
 (0)