realm / realm-js / 7263895097

85%

push

github

Key path filtered notifications in `Object` and `Collection` (#6310)

* Pin Listeners to throwOnReAdd

* Adding key path filtering to Collection

* Tests WIP

* Show the entire diff when deep.equals fails

* Implemented key-path notifications for RealmObject

* Adding an entry to the changelog

* Passing key-paths through for Dictionary

* Adding dictionary tests

* Adding changelog entry about the dictionary fix

* Adding a note on the breaking change

* Apply suggestions from code review

Co-authored-by: LJ <81748770+elle-j@users.noreply.github.com>

* Adding string overload to Object#addListener

* Adding @param doc strings

* Adding an example to the changelog and doc comments.

* Refactored to use a common "expectNotifications" function

* Moved keyPaths before changesAndActions

* Apply suggestions from code review

Co-authored-by: LJ <81748770+elle-j@users.noreply.github.com>

* Adding a few more tests

* Adding a test using wildcard (*)

* Using key-path array of strings in tests again

* Adding multiple listeners with different key paths to verify that the union is indeed used

* Separated key-path tests into separate "it"s

* Update SDK unit tests

* Moved changelog entry from breaking changes to fixed

* Update CHANGELOG.md

* Updated doc comments

* Updated readd tests

* Updating a comment

---------

Co-authored-by: LJ <81748770+elle-j@users.noreply.github.com>

934 of 1146 branches covered (0.0%)

Branch coverage included in aggregate %.

25 of 31 new or added lines in 6 files covered. (80.65%)

2364 of 2696 relevant lines covered (87.69%)

1022.2 hits per line