Thanks to visit codestin.com
Credit goes to github.com

Skip to content

fix: add regression test for optional qdrant-store metadata#138

Open
itsjawreal wants to merge 1 commit into
qdrant:masterfrom
itsjawreal:itsjawreal-patch-20260507054840
Open

fix: add regression test for optional qdrant-store metadata#138
itsjawreal wants to merge 1 commit into
qdrant:masterfrom
itsjawreal:itsjawreal-patch-20260507054840

Conversation

@itsjawreal
Copy link
Copy Markdown

Summary

Add a focused regression test for qdrant-store so storing content without a metadata argument stays supported. The test drives the server through a real FastMCP client and verifies the connector still receives metadata=None.

Why it matters

src/mcp_server_qdrant/mcp_server.py already documents that the metadata parameter shape is client-sensitive, especially for MCP clients such as Cursor. Without coverage, a refactor can silently make metadata required or stop passing None, breaking normal store calls from clients that omit the field.

Testing

Added tests/test_mcp_server.py with an async regression test that instantiates QdrantMCPServer against a mocked connector and calls qdrant-store through fastmcp.Client. The assertion checks both the returned confirmation message and that the stored Entry still carries metadata=None.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant