realm / realm-js / 5142820799

85%

push

github

Add Flexible Sync subscribe/unsubscribe APIs (#5772)

* Implement 'subscribe()' w/o 'timeout' option.

* Add initial 'subscribe()' tests.

* Implement 'unsubscribe()'.

* Add initial 'unsubscribe()' tests.

* Refactor 'TimeoutPromise' to optionally not reject on timeout.

* Update 'subscribe()' to handle 'timeout' option.

* Update tests.

* Update 'unsubscribe()' and let 'mutableSubs.remove()' handle the not-found case.

* Replace 'Results' instance field with call to Core.

* Add 'unnamedOnly' param to 'MutableSubscriptionSet.removeAll()'.

* Add test for removing unnamed subscriptions.

* Add CHANGELOG entry.

* Update formatting.

* Update TSDocs.

* Add test for 'subscribe()'.

* Update use of 'SubscriptionsState' to 'SubscriptionSetState'.

* Remove 'unnamedOnly' param and create 'removeUnnamed()' method.

* Treat a subscription with an empty name as named. (Same behavior as v11)

* Mark 'Results.unsubscribe()' as experimental.

* Remove boolean return type from 'Results.unsubscribe()'.

* Update minor formatting in CHANGELOG.

* Store subscription name on 'Results' to unsubscribe correctly.

* Add more tests to 'unsubscribe()'.

* Update minor formatting in test.

* Add comment regarding 'this.timeout()' in test.

* Mark 'subscribe()' as experimental.

* Add clarification to test.

* Change ordering of condition checks.

* Update CHANGELOG.md

Co-authored-by: Kræn Hansen <kraen.hansen@mongodb.com>

---------

Co-authored-by: Kræn Hansen <kraen.hansen@mongodb.com>

846 of 1049 branches covered (80.65%)

Branch coverage included in aggregate %.

2242 of 2549 relevant lines covered (87.96%)

757.85 hits per line