Skip to content

Add $subtract tests#677

Open
PatersonProjects wants to merge 11 commits into
documentdb:mainfrom
PatersonProjects:substract_tests
Open

Add $subtract tests#677
PatersonProjects wants to merge 11 commits into
documentdb:mainfrom
PatersonProjects:substract_tests

Conversation

@PatersonProjects

Copy link
Copy Markdown
Contributor

This pr adds the tests for the $subtract operator. It was originally authored by @mitchell-elholm.

Ref: Issue #9, #286

Signed-off-by: PatersonProjects <keldonhoff@gmail.com>
Signed-off-by: PatersonProjects <keldonhoff@gmail.com>
This reverts commit 014f2bc.

Signed-off-by: PatersonProjects <keldonhoff@gmail.com>
This reverts commit 51a482a.

Signed-off-by: PatersonProjects <keldonhoff@gmail.com>
…-tests

Signed-off-by: PatersonProjects <keldonhoff@gmail.com>
Signed-off-by: PatersonProjects <keldonhoff@gmail.com>
@PatersonProjects PatersonProjects requested a review from a team as a code owner July 10, 2026 16:38
Signed-off-by: PatersonProjects <keldonhoff@gmail.com>
@PatersonProjects PatersonProjects marked this pull request as draft July 10, 2026 16:50
Signed-off-by: PatersonProjects <keldonhoff@gmail.com>
@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 In Progress
Confidence: 0.85 (mixed)

Reasoning

component from path globs (test-coverage); effort from diff stats (1413+0 LOC, 9 files); LLM: Adds new test cases for the $subtract operator to the compatibility test suite, expanding coverage for a functional operator.

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 10, 2026
@PatersonProjects PatersonProjects marked this pull request as ready for review July 10, 2026 17:38

@danielfrankcom danielfrankcom left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

All minor things, largely looks good

Signed-off-by: PatersonProjects <keldonhoff@gmail.com>
Signed-off-by: PatersonProjects <keldonhoff@gmail.com>
Signed-off-by: PatersonProjects <keldonhoff@gmail.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