Skip to content

Commit 31d35d2

Browse files
authored
Remove the test-unit-verbose target (#716)
<!-- Provide a brief summary of your changes --> ## Motivation and Context <!-- Why is this change needed? What problem does it solve? --> As per @domdomegg's comment - #713 (comment) ## How Has This Been Tested? <!-- Have you tested this in a real application? Which scenarios were tested? --> ## Breaking Changes <!-- Will users need to update their code or configurations? --> ## Types of changes <!-- What types of changes does your code introduce? Put an `x` in all the boxes that apply: --> - [ ] Bug fix (non-breaking change which fixes an issue) - [ ] New feature (non-breaking change which adds functionality) - [ ] Breaking change (fix or feature that would cause existing functionality to change) - [ ] Documentation update ## Checklist <!-- Go over all the following points, and put an `x` in all the boxes that apply. --> - [ ] I have read the [MCP Documentation](https://modelcontextprotocol.io) - [ ] My code follows the repository's style guidelines - [ ] New and existing tests pass locally - [ ] I have added appropriate error handling - [ ] I have added or updated documentation as needed ## Additional context <!-- Add any other context, implementation notes, or design decisions --> Signed-off-by: Radoslav Dimitrov <[email protected]>
1 parent 39824fb commit 31d35d2

File tree

1 file changed

+1
-14
lines changed

1 file changed

+1
-14
lines changed

Makefile

Lines changed: 1 addition & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
.PHONY: help build test test-unit test-unit-verbose test-integration test-endpoints test-publish test-all lint lint-fix validate validate-schemas validate-examples check dev-compose clean publisher generate-schema check-schema
1+
.PHONY: help build test test-unit test-integration test-endpoints test-publish test-all lint lint-fix validate validate-schemas validate-examples check dev-compose clean publisher generate-schema check-schema
22

33
# Default target
44
help: ## Show this help message
@@ -46,19 +46,6 @@ test-unit: ## Run unit tests with coverage (requires PostgreSQL)
4646
@docker compose down postgres >/dev/null 2>&1
4747
@echo "✅ Tests complete"
4848

49-
test-unit-verbose: ## Run unit tests with verbose output
50-
@echo "Starting PostgreSQL for unit tests..."
51-
@docker compose up -d postgres 2>&1 | grep -v "Pulling\|Pulled\|Creating\|Created\|Starting\|Started" || true
52-
@echo "Waiting for PostgreSQL to be ready..."
53-
@sleep 3
54-
@echo ""
55-
@echo "Running unit tests (verbose)..."
56-
@go test -v -race -coverprofile=coverage.out -covermode=atomic ./internal/... ./cmd/... 2>&1 | grep -v "ld: warning:" | grep -v "^ld:"
57-
@echo ""
58-
@go tool cover -html=coverage.out -o coverage.html
59-
@echo "✅ Coverage report: coverage.html"
60-
@docker compose down postgres >/dev/null 2>&1
61-
6249
test: ## Run unit tests (use 'make test-all' to run all tests)
6350
@echo "⚠️ Running unit tests only. Use 'make test-all' to run both unit and integration tests."
6451
@$(MAKE) test-unit

0 commit comments

Comments
 (0)