Nimble
Nimble copied to clipboard
Make Nimble compile for watchOS.
- [x] I have read CONTRIBUTING and have done my best to follow them.
Nimble doesn't compile for watchOS, since throwAssertion
uses an exception type that isn't available for watchOS. This has been a problem for a long time and makes it hard to use Quick and Nimble in projects and packages that target watchOS.
While a real solution may be larger undertaking (one such PR already exists), I have created a pull request that disables the part that doesn't compile. Since only throwAssertion
is affected, it's a very small tweak to a single place in the library.
With this tweak, developers will get a descriptive message and have the choice to either just not unit test on watchOS or to disable tests that rely on throwAssertion
. This will at least make the code compile, which will be a huge improvement.
@ikesyo Any progress on this? I'd love to get my PR merged and released, so that I don't have to use my own forks in my various watchOS supporting projects. Lmk if I can help in any way.
Hey @danielsaidi, I merged your linked PR. I'd like to see CI building/running on watchOS, as well as the watchOS targets added to the xcodeproj before we actually release watchOS support. If you're able to, I'd love to see a PR for those.
I'll leave this open as we don't yet have watchOS support.
That's great - thank you! I'd love to take a look at it if I get some time on my hands. If so, I'll write more here. 👍
In the interest of getting v10 out the door sooner, I'm going to move watchOS support to the next major release.
This was done a few months ago, but forgot to close this. Closing it now!