OHHTTPStubs Add support to WatchOS

Checklist

[ ] I've checked that all new and existing tests pass
[ ] I've updated the documentation if necessary
[ ] I've added an entry in the CHANGELOG to credit myself

Description

Motivation and Context

Aug 02 '18 06:08 linksmt

I'm a bit concerned if those changes are enough, as there is on several places code like #if defined(__IPHONE_7_0), that should be extended with __WATCHOS_2_0

Aug 16 '18 07:08 tfonfara

good point.

Aug 17 '18 14:08 AliSoftware

Ping @linksmt about the #if defined(__IPHONE_7_0) remark above 😉

Aug 28 '18 09:08 AliSoftware

I tried to come up with a solution by myself, but things are petty weird. I've added all necessary precompiler defines, but OHHTTPStubsProtocol isn't working properly. canInitWithRequest is being called and also returns true, but neither initWithRequest nor startLoading gets called. I also tried with a simple approach in a fresh project to implement NSURLProtocol but found same behaviour.

Dec 05 '18 09:12 tfonfara

Can you accept merge? Thanks

Mar 30 '19 13:03 linksmt

@linksmt thank you for submitting this!

Everything looks great for the added target, but I think we should probably have some tests to go along with it so we can make sure we don't break it. 😄

Would you be comfortable adding some tests to the watchOS target?

I think it would also be helpful if you update the CHANGELOG.md to give yourself credit and update the original description of this PR.