Skip to content

test: ValkeyDocumentStore using Mixin Test#3019

Merged
davidsbatista merged 5 commits into
mainfrom
test/valkey-use-mixin-test
Mar 23, 2026
Merged

test: ValkeyDocumentStore using Mixin Test#3019
davidsbatista merged 5 commits into
mainfrom
test/valkey-use-mixin-test

Conversation

@davidsbatista

Copy link
Copy Markdown
Contributor

Proposed Changes:

Adding the following tests:

  • CountDocumentsByFilterTest
  • CountUniqueMetadataByFilterTest
  • GetMetadataFieldsInfoTest
  • GetMetadataFieldMinMaxTest
  • GetMetadataFieldUniqueValuesTest

How did you test it?

  • unit tests, integration tests, manual verification, CI tests

Checklist

@github-actions github-actions Bot added integration:valkey type:documentation Improvements or additions to documentation labels Mar 23, 2026
@davidsbatista davidsbatista marked this pull request as ready for review March 23, 2026 08:57
@davidsbatista davidsbatista requested a review from a team as a code owner March 23, 2026 08:57
@davidsbatista davidsbatista requested review from sjrl and removed request for a team March 23, 2026 08:57

@sjrl sjrl left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks good!

@davidsbatista davidsbatista merged commit e97481a into main Mar 23, 2026
9 checks passed
@davidsbatista davidsbatista deleted the test/valkey-use-mixin-test branch March 23, 2026 10:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

integration:valkey type:documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants