I know this looks like a huge PR, but 2000 of the 3500 changes are of StitchCore.xcodeproj which was regenerated in SPM to add the new target (and in the future we won't even be committing this xcodeproj), and much of the rest is based on code we've already written. I know we said we should be making smaller PRs so hopefully this will be the last of the monolithic PRs.
This PR adds the admin SDK as a separate target/product of the StitchCore package, and adds integration tests to the StitchCore-iOS project.
The integration tests include everything that was tested on Android, except custom auth. I have that test commented out for now and will try now to fix it. We'll need CommonCrypto for the test to run properly (for generating the mock JWT) and Jason said that's non-trivial to add for iOS so I wanted to get the PR out first before I spend too much time on that.
Drive-by changes:
AuthInfo now supports responses which are missing a refresh token or device ID, which is the case for a link request.
Fixed the link test so that the mock response is missing the fields
Fixed email confirmation route
publicly exposed MemoryStorage in StitchCore so it could be used by the admin client for tests.
I know this looks like a huge PR, but 2000 of the 3500 changes are of
StitchCore.xcodeprojwhich was regenerated in SPM to add the new target (and in the future we won't even be committing this xcodeproj), and much of the rest is based on code we've already written. I know we said we should be making smaller PRs so hopefully this will be the last of the monolithic PRs.This PR adds the admin SDK as a separate target/product of the
StitchCorepackage, and adds integration tests to theStitchCore-iOSproject.The integration tests include everything that was tested on Android, except custom auth. I have that test commented out for now and will try now to fix it. We'll need
CommonCryptofor the test to run properly (for generating the mock JWT) and Jason said that's non-trivial to add for iOS so I wanted to get the PR out first before I spend too much time on that.Drive-by changes:
MemoryStorageinStitchCoreso it could be used by the admin client for tests.