Skip to content

Add $tsIncrement compatibility tests#654

Open
RyanGarfinkel wants to merge 4 commits into
documentdb:mainfrom
RyanGarfinkel:add-tsIncrement-compatibility-tests
Open

Add $tsIncrement compatibility tests#654
RyanGarfinkel wants to merge 4 commits into
documentdb:mainfrom
RyanGarfinkel:add-tsIncrement-compatibility-tests

Conversation

@RyanGarfinkel

Copy link
Copy Markdown

What does this PR do?

This PR introduces additional compatibility tests for the $tsIncrement operator. This includes null/missing values, invalid types, boundary values, and nested field access. Wiring tests were also added to verify functionality in $match, $expr, and $group expressions. An additional error code, TS_INCREMENT_TYPE_ERROR was added specifically for MongoDB type mismatches.

Why was this PR needed?

This PR Closes #210, which required adding additional test coverage to the $tsIncrement operator. Initially, this operator only had one smoke test, which tested basic functionality, not including null/missing input and invalid types.

What are the relevant issue numbers?

Closes #210

Does this PR meet the acceptance criteria?

  • Tests added for new/changed behavior
  • All tests passing
  • Follows project style guide
  • No breaking changes introduced (new tests added)
  • Documentation updated (if applicable)

@RyanGarfinkel RyanGarfinkel requested a review from a team as a code owner June 29, 2026 04:14
@RyanGarfinkel RyanGarfinkel force-pushed the add-tsIncrement-compatibility-tests branch from 750e3be to 9e59c4e Compare June 29, 2026 04:18
Signed-off-by: RyanGarfinkel <113050972+RyanGarfinkel@users.noreply.github.com>
Signed-off-by: RyanGarfinkel <113050972+RyanGarfinkel@users.noreply.github.com>
Signed-off-by: RyanGarfinkel <113050972+RyanGarfinkel@users.noreply.github.com>
@RyanGarfinkel RyanGarfinkel force-pushed the add-tsIncrement-compatibility-tests branch from 9e59c4e to 3e5f020 Compare June 29, 2026 04:23
@documentdb-triage-tool documentdb-triage-tool Bot added compatibility test Compatibility test related enhancement New feature or request labels Jun 29, 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 M · Status Needs Review
Confidence: 0.30 (deterministic)

Reasoning

component from path globs (test-coverage, test-framework); effort from diff stats (267+0 LOC, 4 files); LLM failed: Invalid response body while trying to fetch https://api.anthropic.com/v1/messages: Premature close

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.

Add compatibility test for $tsIncrement (second pass)

1 participant