Fix #41 unable to integrate test target

joshuatbrown commented 6 years ago

This fixes #41. However, due to the way we fixed it, the Tests target in RZBluetooth no longer compiles. We've tried various ways to fix the problem but haven't found a solution yet. I'm hoping that by opening this PR, we'll get some help with the Tests target.

Here's what we did:

created a new podspec for RZBluetoothTest that contains the files that used to be part of the RZBluetooth/Test subspec
changed the imports in RZBSimulatedTestCase since they now depend on the main RZBluetooth/Mock spec (and therefore need to be imported with RZBluetooth/...)

The second bullet is where the problem lies. By importing with RZBluetooth/..., we've broken the test target which does not have those files in a separate framework. But importing this way is required so that the new RZBluetoothTest podspec builds in CocoaPods.

mplorentz commented 5 years ago

@KingOfBrian I think I made all the changes you requested. I also updated the RZBluetoothExample app to use the RZBluetoothTest podspec. Can you re-review?

joshbrown-eg commented 5 years ago

This looks good — would love to get this merged.

cpatterson-lilly commented 5 years ago

Does RZBluetoothMock need similar treatment to be broken out into its own podspec?

Rightpoint / RZBluetooth

Fix #41 unable to integrate test target #92