add support for mergeObjects operator by elbehery95 · Pull Request #689 · mongomock/mongomock

Conversation

@elbehery95

As per #642

The current implementation follows the operator as defined such that on all null an empty doc is returned, and the docs are merged using dict.update method

pcorpet

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for your PR.

Please add multiple tests in test__mongomock to ensure there are no regressions in the future.

@elbehery95

removed mergeobjects exception checking tests and added a single test__merge_objects under the aggregate tests

@pcorpet

@codecov

@pcorpet

2 participants

@elbehery95 @pcorpet