wordpress-mobile / WordPressAuthenticator-iOS

GNU General Public License v2.0
17 stars 11 forks source link

Replace `GIDGoogleUser` with `SocialService.User` in case .google #764

Closed mokagio closed 1 year ago

mokagio commented 1 year ago

Closes #759.

As described in #759, the idea with this PR is to remove the Google SDK-dependent type from the SocialService google case associated value and use the pure Swift SocialService.User instead. The new type was introduced in #763.

To test

The code changes here are consumed via unit tests alone. The demo app doesn't read the loginFields metadata where SocialService lives.

The "real" test for this new code is in the Jetpack/WordPress iOS that uses it https://github.com/wordpress-mobile/WordPress-iOS/pull/20128