Skip to content

Add collMod command tests#626

Merged
eerxuan merged 8 commits into
documentdb:mainfrom
danielfrankcom:pr/collMod
Jul 2, 2026
Merged

Add collMod command tests#626
eerxuan merged 8 commits into
documentdb:mainfrom
danielfrankcom:pr/collMod

Conversation

@danielfrankcom

@danielfrankcom danielfrankcom commented Jun 18, 2026

Copy link
Copy Markdown
Collaborator

This change adds tests for the collMod collection command.

Closes #350

Signed-off-by: Daniel Frankcom <frankcom@amazon.com>
@danielfrankcom danielfrankcom requested a review from a team as a code owner June 18, 2026 23:28
@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 XL · Status Needs Review
Confidence: 0.85 (mixed)

Reasoning

component from path globs (test-coverage, test-framework); effort from diff stats (6398+0 LOC, 25 files); LLM: Adds new test coverage for the collMod command, expanding compatibility test suite 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 Jun 19, 2026
Signed-off-by: Daniel Frankcom <frankcom@amazon.com>
Signed-off-by: Daniel Frankcom <frankcom@amazon.com>
Signed-off-by: Daniel Frankcom <frankcom@amazon.com>
@danielfrankcom danielfrankcom requested a review from tonglyu June 22, 2026 17:39
Signed-off-by: Daniel Frankcom <frankcom@amazon.com>
Introduce the oplog capability (a replicated local.oplog.rs exists) in the precondition table and migrate the collMod tests from replica_set/standalone markers to requires(...) markers: oplog for the oplog.rs capped-collection cases, change_streams for the $changeStream view-pipeline cases, and quorum_write_concern for the quorum writeConcern cases.

Signed-off-by: Daniel Frankcom <frankcom@amazon.com>
Signed-off-by: Daniel Frankcom <frankcom@amazon.com>
@eerxuan eerxuan merged commit 182f758 into documentdb:main Jul 2, 2026
20 checks passed
krishnasai453 pushed a commit to krishnasai453/functional-tests that referenced this pull request Jul 8, 2026
Signed-off-by: Daniel Frankcom <frankcom@amazon.com>
SarthakDalmia1 pushed a commit to SarthakDalmia1/functional-tests that referenced this pull request Jul 10, 2026
Signed-off-by: Daniel Frankcom <frankcom@amazon.com>
Signed-off-by: Sarthak Dalmia <sadalmia@microsoft.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 collMod

3 participants