Added behaviour test for $pull,$pop,$max,$[<identifier>] - #562
Conversation
|
🤖 Auto-triaged by documentdb-triage-tool. Applied: Reasoningcomponent from path globs (test-coverage); effort from diff stats (1318+0 LOC, 4 files); LLM: Adds comprehensive compatibility/behavior test coverage for multiple update operators ($max, $pop, If a label is wrong, remove it manually and ping |
Signed-off-by: PatersonProjects <[email protected]> Signed-off-by: afanankhan <[email protected]>
Comprehensive compat-functional tests covering type coercion, boundary values, nested paths, command paths (update, findAndModify), multi-doc updates, upsert, composition, and error cases. 3 known divergences tracked with engine_xfail markers: - $[] on empty array: nModified=1 instead of 0 (ADO #5329571) - $[] on non-array field: error code 28 instead of 2 (ADO #5329572) - $ positional nested array: matches wrong element (ADO #5329573) Co-authored-by: Copilot <[email protected]> Signed-off-by: afanankhan <[email protected]>
Comprehensive compat-functional tests covering: - $max: type comparison, cross-type ordering, NaN, dates, missing fields - $pop: first/last, empty array, nested paths, invalid values - $pull: exact match, conditional ($gte, $in), document match, nested - $[<identifier>]: basic filters, scalar arrays, multi, findAndModify 2 divergences tracked with engine_xfail markers: - $pop on non-array: error code 2 instead of 14 (TypeMismatch) - $[<identifier>] invalid identifier: accepts instead of rejecting All tests stable across 5 serial iterations. Co-authored-by: Copilot <[email protected]> Signed-off-by: afanankhan <[email protected]>
This reverts commit 851e883. Signed-off-by: afanankhan <[email protected]>
Signed-off-by: afanankhan <[email protected]>
Signed-off-by: afanankhan <[email protected]>
d946778 to
77c6f8e
Compare
eerxuan
left a comment
There was a problem hiding this comment.
Please claim the tracking issue before raise PR, we are seeing overlapping effort across PRs. Also can you start with issues with "Second pass"? Amazon team is working through First pass features. We planned 48 features in this sprint, will claim the issues shortly.
Add behavior tests for $max, $pop,$pull, $ []