adityamparikh opened a new pull request, #91: URL: https://github.com/apache/solr-mcp/pull/91
## Summary - Adds `McpClientIntegrationTest` that exercises the MCP server through a real MCP client over HTTP - Reorganizes test naming: `*ServiceTest.java` = unit tests (mocks), `*ServiceIntegrationTest.java` = integration tests (Testcontainers) - Removes redundant `*DirectTest.java` files ## Test coverage (McpClientIntegrationTest) - Protocol: ping, tool discovery (all 7+ tools validated) - Collections: create, list, health check, stats, schema - Indexing: JSON documents (5 docs), CSV documents (2 docs) - Search: all docs, filter query, keyword, pagination, facets - Verifies document counts after each indexing step (5 → 7) ## Test reorganization - `SearchServiceTest` = unit tests (mocked SolrClient), `SearchServiceIntegrationTest` = real Solr - `IndexingServiceTest` = unit tests (mocked SolrClient), `IndexingServiceIntegrationTest` = real Solr - Deleted `SearchServiceDirectTest` and `IndexingServiceDirectTest` (redundant with the above) - Fixed `TestcontainersConfiguration` default Solr image to `solr:9.9-slim` ## Test plan - [x] `./gradlew build` passes with no regressions 🤖 Generated with [Claude Code](https://claude.com/claude-code) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
