Remove no-longer-necessary params from NewGoogleAuthenticator inits

See work done in the previous two commits.

Author: mokagio

Demo/AuthenticatorDemo/ViewController.swift

@@ -35,7 +35,6 @@ class ViewController: UIViewController {
         clientId: GoogleClientId(string: APICredentials.googleLoginClientId)!,
         scheme: APICredentials.googleLoginSchemeId,
         audience: APICredentials.googleLoginServerClientId,
-        viewController: self,
         urlSession: URLSession.shared
     )
 

WordPressAuthenticator/GoogleSignIn/NewGoogleAuthenticator.swift

@@ -5,38 +5,18 @@ public class NewGoogleAuthenticator: NSObject {
     let clientId: GoogleClientId
     let scheme: String
     let audience: String
-    let contextProvider: ASWebAuthenticationPresentationContextProviding
-
     let oauthTokenGetter: GoogleOAuthTokenGetting
 
     public convenience init(
         clientId: GoogleClientId,
         scheme: String,
         audience: String,
-        viewController: UIViewController,
-        urlSession: URLSession
-    ) {
-        self.init(
-            clientId: clientId,
-            scheme: scheme,
-            audience: audience,
-            contextProvider: WebAuthenticationPresentationContext(viewController: viewController),
-            oautTokenGetter: GoogleOAuthTokenGetter(dataGetter: urlSession)
-        )
-    }
-
-    public convenience init(
-        clientId: GoogleClientId,
-        scheme: String,
-        audience: String,
-        contextProvider: ASWebAuthenticationPresentationContextProviding,
         urlSession: URLSession
     ) {
         self.init(
             clientId: clientId,
             scheme: scheme,
             audience: audience,
-            contextProvider: contextProvider,
             oautTokenGetter: GoogleOAuthTokenGetter(dataGetter: urlSession)
         )
     }
@@ -45,14 +25,12 @@ public class NewGoogleAuthenticator: NSObject {
         clientId: GoogleClientId,
         scheme: String,
         audience: String,
-        contextProvider: ASWebAuthenticationPresentationContextProviding,
         oautTokenGetter: GoogleOAuthTokenGetting
     ) {
         self.clientId = clientId
         self.scheme = scheme
         self.audience = audience
         self.oauthTokenGetter = oautTokenGetter
-        self.contextProvider = contextProvider
     }
 
     /// Get the user's OAuth token from their Google account. This token can be used to authenticate with the WordPress backend.

WordPressAuthenticator/Unified Auth/GoogleAuthenticator.swift

@@ -307,14 +307,10 @@ extension GoogleAuthenticator {
 
         trackRequestAuthorizitation(type: authType)
 
-        // We might want to change this in subsequent iterations, perhaps by moving the
-        // `contextProvider` to the `getOAuthToken()` method so to allow it to be stored
-        // as a `lazy` property?
         let sdkLessGoogleAuthenticator = NewGoogleAuthenticator(
             clientId: authConfig.googleClientId,
             scheme: authConfig.googleLoginScheme,
             audience: authConfig.googleLoginServerClientId,
-            contextProvider: WebAuthenticationPresentationContext(viewController: viewController),
             urlSession: .shared
         )
 

WordPressAuthenticatorTests/GoogleSignIn/NewGoogleAuthenticatorTests.swift

@@ -8,11 +8,10 @@ class NewGoogleAuthenticatorTests: XCTestCase {
     func testRequestingOAuthTokenThrowsIfCodeCannotBeExtractedFromURL() async throws {
         // Notice the use of a stub that returns a successful value.
         // This way, if we get an error, we can be more confident it's legit.
-        let authenticator = await NewGoogleAuthenticator(
+        let authenticator = NewGoogleAuthenticator(
             clientId: fakeClientId,
             scheme: "scheme",
             audience: "audience",
-            contextProvider: FakeContextProvider(),
             oautTokenGetter: GoogleOAuthTokenGettingStub(response: .fixture())
         )
         let url = URL(string: "https://test.com?without=code")!
@@ -35,11 +34,10 @@ class NewGoogleAuthenticatorTests: XCTestCase {
     }
 
     func testRequestingOAuthTokenRethrowsTheErrorItRecives() async throws {
-        let authenticator = await NewGoogleAuthenticator(
+        let authenticator = NewGoogleAuthenticator(
             clientId: fakeClientId,
             scheme: "scheme",
             audience: "audience",
-            contextProvider: FakeContextProvider(),
             oautTokenGetter: GoogleOAuthTokenGettingStub(error: TestError(id: 1))
         )
         let url = URL(string: "https://test.com?code=a_code")!
@@ -59,11 +57,10 @@ class NewGoogleAuthenticatorTests: XCTestCase {
     }
 
     func testRequestingOAuthTokenThrowsIfIdTokenMissingFromResponse() async throws {
-        let authenticator = await NewGoogleAuthenticator(
+        let authenticator = NewGoogleAuthenticator(
             clientId: fakeClientId,
             scheme: "scheme",
             audience: "audience",
-            contextProvider: FakeContextProvider(),
             oautTokenGetter: GoogleOAuthTokenGettingStub(response: .fixture(rawIDToken: .none))
         )
         let url = URL(string: "https://test.com?code=a_code")!
@@ -85,11 +82,10 @@ class NewGoogleAuthenticatorTests: XCTestCase {
     }
 
     func testRequestingOAuthTokenReturnsTokenIfSuccessful() async throws {
-        let authenticator = await NewGoogleAuthenticator(
+        let authenticator = NewGoogleAuthenticator(
             clientId: fakeClientId,
             scheme: "scheme",
             audience: "audience",
-            contextProvider: FakeContextProvider(),
             oautTokenGetter: GoogleOAuthTokenGettingStub(response: .fixture(rawIDToken: JSONWebToken.validJWTStringWithEmail))
         )
         let url = URL(string: "https://test.com?code=a_code")!
@@ -107,11 +103,3 @@ class NewGoogleAuthenticatorTests: XCTestCase {
         }
     }
 }
-
-import AuthenticationServices
-
-class FakeContextProvider: UIViewController, ASWebAuthenticationPresentationContextProviding {
-    func presentationAnchor(for session: ASWebAuthenticationSession) -> ASPresentationAnchor {
-        return view.window!
-    }
-}