ably / ably-asset-tracking-swift

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

Extend `Blocking.run`’s interruption behaviour to work across threads #621

Closed lawrence-forooghian closed 1 year ago

lawrence-forooghian commented 1 year ago

This PR is based on top of #618; please review that one first.

Another thing we need (well, "need" might be a stretch, let’s say a "thing that solves a problem that’s entirely possibly of my own making"; see next paragraph) for the subscriber network connectivity tests.

I would not be at all surprised to find that there is a simpler way of writing the tests that doesn’t require this vile thing, but this is the solution I have, and it seems to work, and I don’t really feel like investing more time in these tests.

I wouldn't recommend spending too much time trying to understand this PR unless you really think you'd enjoy it. It's test suite code, at the end of the day.