Skip to content

Add verbose syntax tests for $lookup#699

Open
danielfrankcom wants to merge 1 commit into
documentdb:mainfrom
danielfrankcom:pr/lookup-verbose
Open

Add verbose syntax tests for $lookup#699
danielfrankcom wants to merge 1 commit into
documentdb:mainfrom
danielfrankcom:pr/lookup-verbose

Conversation

@danielfrankcom

Copy link
Copy Markdown
Collaborator

This change adds testing for the verbose (let and pipeline) form of $lookup.

It focuses on how let variables are resolved across the sub-pipeline, including how each supported stage recognizes them, how they scope across nested $lookup stages, and how they remain accessible across multi-stage sub-pipelines.

The new organization structure replaces the earlier test_lookup_with_expr.py file.

This is related to #673 and the comments there. Changes based on @PatersonProjects' original PR.

Signed-off-by: Daniel Frankcom <frankcom@amazon.com>
@danielfrankcom
danielfrankcom requested a review from a team as a code owner July 22, 2026 19:33
@documentdb-triage-tool documentdb-triage-tool Bot added compatibility test Compatibility test related enhancement New feature or request labels Jul 22, 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.88 (mixed)

Reasoning

component from path globs (test-coverage); effort from diff stats (1590+309 LOC, 6 files); LLM: Adds new test coverage for the verbose $lookup syntax (let/pipeline), expanding compatibility test suite without touching production code.

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.

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.

1 participant