Skip to content

Add $exp expression tests#663

Open
danielfrankcom wants to merge 3 commits into
documentdb:mainfrom
danielfrankcom:pr/exp
Open

Add $exp expression tests#663
danielfrankcom wants to merge 3 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>
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

2 participants