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 <keldonhoff@gmail.com> Signed-off-by: afanankhan <afanankhan@microsoft.com>
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 <223556219+Copilot@users.noreply.github.com> Signed-off-by: afanankhan <afanankhan@microsoft.com>
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 <223556219+Copilot@users.noreply.github.com> Signed-off-by: afanankhan <afanankhan@microsoft.com>
This reverts commit 851e883. Signed-off-by: afanankhan <afanankhan@microsoft.com>
Signed-off-by: afanankhan <afanankhan@microsoft.com>
Signed-off-by: afanankhan <afanankhan@microsoft.com>
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, $ []