Skip to content

Add $bucketAuto _id override and reshaped-upstream compatibility tests - #721

Merged
eerxuan merged 1 commit into
documentdb:mainfrom
jingyaog:test/bucketAuto-id-override-reshaped-upstream
Aug 21, 2026
Merged

Add $bucketAuto _id override and reshaped-upstream compatibility tests#721
eerxuan merged 1 commit into
documentdb:mainfrom
jingyaog:test/bucketAuto-id-override-reshaped-upstream

Conversation

@jingyaog

Copy link
Copy Markdown
Contributor

Summary

  • Add _id-in-output override tests for $bucketAuto (accumulator overrides the
    default {min, max} boundary; non-accumulator _id is rejected with 40234) —
    contrasting $bucket, which reserves _id.
  • Add reshaped/re-materialized-upstream tests: $bucketAuto after $unwind,
    $project, $addFields, $unset, $replaceRoot (plus a pass-through $match
    control), with wide docs and buckets > distinct groupBy values.
  • Generalize both cases as coverage guidelines in TEST_COVERAGE.md §15.

Test plan

All new tests pass against the reference server (MongoDB 8.2.4, the target spec version).

@jingyaog
jingyaog requested a review from a team as a code owner August 20, 2026 19:30
@documentdb-triage-tool documentdb-triage-tool Bot added compatibility test Compatibility test related documentation Improvements or additions to documentation enhancement New feature or request labels Aug 20, 2026
@documentdb-triage-tool

Copy link
Copy Markdown

🤖 Auto-triaged by documentdb-triage-tool.

Applied: compatibility test, documentation, enhancement
Project fields suggested: Component test-coverage · Priority P2 · Effort M · Status Needs Review
Confidence: 0.85 (mixed)

Reasoning

component from path globs (test-coverage, docs); effort from diff stats (204+1 LOC, 4 files); LLM: Adds new compatibility test cases for $bucketAuto edge cases and updates TEST_COVERAGE.md, touching test files and docs within a single component.

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.

Comment thread docs/testing/TEST_COVERAGE.md Outdated
@jingyaog
jingyaog force-pushed the test/bucketAuto-id-override-reshaped-upstream branch from 217c846 to c1f33a9 Compare August 21, 2026 22:00
@jingyaog
jingyaog force-pushed the test/bucketAuto-id-override-reshaped-upstream branch from c1f33a9 to ecef501 Compare August 21, 2026 22:17
@eerxuan
eerxuan merged commit 51bf29f into documentdb:main Aug 21, 2026
20 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

compatibility test Compatibility test related documentation Improvements or additions to documentation enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants