Skip to content

Add $exp expression tests#663

Merged
eerxuan merged 4 commits into
documentdb:mainfrom
danielfrankcom:pr/exp
Jul 7, 2026
Merged

Add $exp expression tests#663
eerxuan merged 4 commits into
documentdb:mainfrom
danielfrankcom:pr/exp

Conversation

@danielfrankcom

Copy link
Copy Markdown
Collaborator

This change adds tests for the $exp expression. It was originally authored by @vic-tsang.

Closes #275

This change adds tests for the $exp expression. It was originally authored
by @vic-tsang.

Closes documentdb#275

Co-authored-by: Victor Tsang <vitsangp@amazon.com>
Signed-off-by: Daniel Frankcom <frankcom@amazon.com>
@danielfrankcom danielfrankcom requested a review from a team as a code owner July 1, 2026 22:05
@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.85 (mixed)

Reasoning

component from path globs (test-coverage); effort from diff stats (638+0 LOC, 7 files); LLM: Adds new compatibility tests for the $exp expression operator, expanding test coverage for an existing feature.

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.

@documentdb-triage-tool documentdb-triage-tool Bot added compatibility test Compatibility test related enhancement New feature or request labels Jul 1, 2026
@PatersonProjects

PatersonProjects commented Jul 2, 2026

Copy link
Copy Markdown
Contributor

Missing __init__.py

@PatersonProjects PatersonProjects 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.

Could implement a nested field test
doc={"a": {"b": 1}}, expression={"$exp": "$a.b"}

Signed-off-by: Daniel Frankcom <frankcom@amazon.com>
Signed-off-by: Daniel Frankcom <frankcom@amazon.com>
@eerxuan eerxuan merged commit a06c411 into documentdb:main Jul 7, 2026
20 checks passed
krishnasai453 pushed a commit to krishnasai453/functional-tests that referenced this pull request Jul 8, 2026
Signed-off-by: Daniel Frankcom <frankcom@amazon.com>
SarthakDalmia1 pushed a commit to SarthakDalmia1/functional-tests that referenced this pull request Jul 10, 2026
Signed-off-by: Daniel Frankcom <frankcom@amazon.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.

Add compatibility test for $exp

3 participants