Added update tests for setOnInsert#576
Conversation
Signed-off-by: Victor [C] Tsang <vitsangp@amazon.com>
|
🤖 Auto-triaged by documentdb-triage-tool. Applied: Reasoningcomponent from path globs (test-coverage, test-framework); effort from diff stats (952+0 LOC, 10 files); LLM: Adds new update operator test cases for $setOnInsert, expanding test coverage for an existing feature. If a label is wrong, remove it manually and ping |
Signed-off-by: Victor Tsang <vitsangp@amazon.com>
| @@ -0,0 +1,126 @@ | |||
| """ | |||
There was a problem hiding this comment.
| msg="Should preserve Decimal128 -0", | ||
| ), | ||
| UpdateTestCase( | ||
| id="null_vs_missing", |
There was a problem hiding this comment.
[Minor] The case ID and message claim a "null vs missing" distinction, but the assertion only verifies field: None is present — it doesn't actually exercise the missing path. Either rename to preserves_null (drop the "vs missing" framing), or add a sibling test using {$exists: True} filter to prove the field is stored as null and not missing.
This PR contains:
Ref: