Skip to content

test: add Qdrant adapter tests - #109

Merged
AnupamKumar-1 merged 1 commit into
Obyflow:mainfrom
OMSHIVSHARAN:feat/qdrant-adapter-tests
Sep 20, 2026
Merged

AnupamKumar-1 merged 1 commit into
Obyflow:mainfrom
OMSHIVSHARAN:feat/qdrant-adapter-tests

Conversation

@OMSHIVSHARAN

@OMSHIVSHARAN OMSHIVSHARAN commented Sep 20, 2026 •

Copy link
Copy Markdown
Contributor

Summary

Adds tests for the Qdrant vector database adapter.

Changes

  • Add tests for instrumentQdrantClient
  • Cover search and emitted vector_op query events
  • Cover upsert and emitted vector_op events
  • Cover delete and emitted vector_op events
  • Verify return values are passed through correctly

Testing

  • pnpm --filter @obyflow/adapter-vectordb test
  • 5 test files passed
  • 12 tests passed
  • No regressions

Related Issue

Part of #68

Co-Authored-By: Claude Code <noreply@anthropic.com>

@AnupamKumar-1 AnupamKumar-1 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks @OMSHIVSHARAN! Pulled the branch and ran it locally: the 3 new Qdrant tests pass, and the build and lint are clean. It follows the pinecone.test.ts template and covers search, upsert and delete, plus the return-value passthrough.

Note: this only covers Qdrant. #68 also lists chroma.ts, milvus.ts and weaviate.ts, which still have no tests. I've changed the PR description to "Part of #68" so the issue stays open until those are done. Happy for them to come as separate PRs.

@AnupamKumar-1
AnupamKumar-1 merged commit 2058d58 into Obyflow:main Sep 20, 2026
5 checks passed
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.

2 participants