Skip to content

Add $planCacheListFilters command tests#588

Merged
eerxuan merged 12 commits into
documentdb:mainfrom
alinaliBQ:planCacheListFilters
Jul 2, 2026
Merged

Add $planCacheListFilters command tests#588
eerxuan merged 12 commits into
documentdb:mainfrom
alinaliBQ:planCacheListFilters

Conversation

@alinaliBQ

Copy link
Copy Markdown
Contributor

This change adds tests for the $planCacheListFilters command operator.

Add command operator tests for $planCacheListFilters. Tests database $planCacheListFilters behavior, syntax, and expected errors.

This PR adds setup field in CommandTestCase.

@alinaliBQ alinaliBQ force-pushed the planCacheListFilters branch from b86c43c to 1ec7016 Compare June 9, 2026 23:47
@alinaliBQ alinaliBQ marked this pull request as ready for review June 9, 2026 23:48
@alinaliBQ alinaliBQ requested a review from a team as a code owner June 9, 2026 23:48
@documentdb-triage-tool documentdb-triage-tool Bot added compatibility test Compatibility test related enhancement New feature or request labels Jun 10, 2026
@documentdb-triage-tool

Copy link
Copy Markdown

🤖 Auto-triaged by documentdb-triage-tool.

Applied: compatibility test, enhancement
Project fields suggested: Component test-coverage · Priority P2 · Effort L · Status Needs Review
Confidence: 0.82 (mixed)

Reasoning

component from path globs (test-coverage); effort from diff stats (1031+0 LOC, 6 files); LLM: Adds new functional test coverage for the $planCacheListFilters command, including a minor test framework extension (adding a setup field to CommandTestCase).

If a label is wrong, remove it manually and ping @patty-chow so the rules can be tuned. The bot will not re-label items that already have component labels.

@alinaliBQ alinaliBQ left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

@Rishabh998 comments are addressed

@alinaliBQ alinaliBQ force-pushed the planCacheListFilters branch from 4048d40 to 3dff088 Compare June 12, 2026 21:42
alinaliBQ added 11 commits June 22, 2026 11:36
Signed-off-by: Alina (Xi) Li <Alina.Li@improving.com>
Signed-off-by: Alina (Xi) Li <Alina.Li@improving.com>
Signed-off-by: Alina (Xi) Li <Alina.Li@improving.com>
Signed-off-by: Alina (Xi) Li <Alina.Li@improving.com>
Signed-off-by: Alina (Xi) Li <Alina.Li@improving.com>
Signed-off-by: Alina (Xi) Li <Alina.Li@improving.com>
Signed-off-by: Alina (Xi) Li <Alina.Li@improving.com>
Signed-off-by: Alina (Xi) Li <Alina.Li@improving.com>
Signed-off-by: Alina (Xi) Li <Alina.Li@improving.com>
Signed-off-by: Alina (Xi) Li <Alina.Li@improving.com>
Signed-off-by: Alina (Xi) Li <Alina.Li@improving.com>
@alinaliBQ alinaliBQ force-pushed the planCacheListFilters branch from 3dff088 to 733c807 Compare June 22, 2026 18:37
@eerxuan eerxuan merged commit 369f18c into documentdb:main Jul 2, 2026
14 checks passed
krishnasai453 pushed a commit to krishnasai453/functional-tests that referenced this pull request Jul 8, 2026
Signed-off-by: Alina (Xi) Li <Alina.Li@improving.com>
SarthakDalmia1 pushed a commit to SarthakDalmia1/functional-tests that referenced this pull request Jul 10, 2026
Signed-off-by: Alina (Xi) Li <Alina.Li@improving.com>
Signed-off-by: Sarthak Dalmia <sadalmia@microsoft.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

compatibility test Compatibility test related enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants