ably / ably-asset-tracking-swift

iOS client SDKs for the Ably Asset Tracking service.
Apache License 2.0
9 stars 6 forks source link

Do not emit a resolution to subscriber for `ABSENT` or `LEAVE` presence messages #633

Closed lawrence-forooghian closed 1 year ago

lawrence-forooghian commented 1 year ago

This mimics the behaviour of Android’s updateForPresenceMessagesAndThenEmitStateEventsIfRequired at commit 4d13ae7.

I’ve only added this piecemeal change to get the Null* subscriber network connectivity tests to pass (see this comment). We’ll be removing this code soon enough as part of our Android parity work, so I haven’t bothered changing the tests since this part of the code is currently untested anyway.