Skip to content

test: PGVectorDocumentStoree use Mixin tests#3003

Merged
davidsbatista merged 4 commits intomainfrom
test/pgvector-use-mixin-tests
Mar 20, 2026
Merged

test: PGVectorDocumentStoree use Mixin tests#3003
davidsbatista merged 4 commits intomainfrom
test/pgvector-use-mixin-tests

Conversation

@davidsbatista
Copy link
Copy Markdown
Contributor

Proposed Changes:

  • Use Mixin tests in PGVectorDocumentStore

How did you test it?

unit tests, integration tests, manual verification + CI

Checklist

@davidsbatista davidsbatista requested a review from a team as a code owner March 20, 2026 16:37
@davidsbatista davidsbatista requested review from anakin87 and removed request for a team March 20, 2026 16:37
@github-actions github-actions Bot added the type:documentation Improvements or additions to documentation label Mar 20, 2026
Copy link
Copy Markdown
Member

@anakin87 anakin87 left a comment

Choose a reason for hiding this comment

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

It seems that old tests have not been removed, no?

@davidsbatista
Copy link
Copy Markdown
Contributor Author

It seems that old tests have not been removed, no?

missed that, thanks for double checking

@davidsbatista davidsbatista requested a review from anakin87 March 20, 2026 17:11
@davidsbatista davidsbatista merged commit 7de3e79 into main Mar 20, 2026
9 checks passed
@davidsbatista davidsbatista deleted the test/pgvector-use-mixin-tests branch March 20, 2026 17:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

integration:pgvector type:documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants