[google_sign_in] Add Android account name field as optional (implementation package changes) (flutter#8805)

Implementation package changes for [PR 8573](flutter#8573). This PR adds support for a new passthrough field to explicitly set an account name while signing in for Android devices.

Flutter issue: [163257](flutter/flutter#163257)

Author: neilself

packages/google_sign_in/google_sign_in_android/CHANGELOG.md

@@ -1,3 +1,7 @@
+## 6.2.0
+
+* Adds a sign-in field to allow clients to explicitly specify an account name.
+
 ## 6.1.36
 
 * Updates compileSdk 34 to flutter.compileSdkVersion.

packages/google_sign_in/google_sign_in_android/android/src/main/java/io/flutter/plugins/googlesignin/GoogleSignInPlugin.java

@@ -255,6 +255,11 @@ public void init(@NonNull Messages.InitParams params) {
           optionsBuilder.setHostedDomain(params.getHostedDomain());
         }
 
+        String forceAccountName = params.getForceAccountName();
+        if (!isNullOrEmpty(forceAccountName)) {
+          optionsBuilder.setAccountName(forceAccountName);
+        }
+
         signInClient = googleSignInWrapper.getClient(context, optionsBuilder.build());
       } catch (Exception e) {
         throw new FlutterError(ERROR_REASON_EXCEPTION, e.getMessage(), null);

packages/google_sign_in/google_sign_in_android/android/src/main/java/io/flutter/plugins/googlesignin/Messages.java

@@ -156,6 +156,16 @@ public void setForceCodeForRefreshToken(@NonNull Boolean setterArg) {
       this.forceCodeForRefreshToken = setterArg;
     }
 
+    private @Nullable String forceAccountName;
+
+    public @Nullable String getForceAccountName() {
+      return forceAccountName;
+    }
+
+    public void setForceAccountName(@Nullable String setterArg) {
+      this.forceAccountName = setterArg;
+    }
+
     /** Constructor is non-public to enforce null safety; use Builder. */
     InitParams() {}
 
@@ -173,13 +183,20 @@ public boolean equals(Object o) {
           && Objects.equals(hostedDomain, that.hostedDomain)
           && Objects.equals(clientId, that.clientId)
           && Objects.equals(serverClientId, that.serverClientId)
-          && forceCodeForRefreshToken.equals(that.forceCodeForRefreshToken);
+          && forceCodeForRefreshToken.equals(that.forceCodeForRefreshToken)
+          && Objects.equals(forceAccountName, that.forceAccountName);
     }
 
     @Override
     public int hashCode() {
       return Objects.hash(
-          scopes, signInType, hostedDomain, clientId, serverClientId, forceCodeForRefreshToken);
+          scopes,
+          signInType,
+          hostedDomain,
+          clientId,
+          serverClientId,
+          forceCodeForRefreshToken,
+          forceAccountName);
     }
 
     public static final class Builder {
@@ -232,6 +249,14 @@ public static final class Builder {
         return this;
       }
 
+      private @Nullable String forceAccountName;
+
+      @CanIgnoreReturnValue
+      public @NonNull Builder setForceAccountName(@Nullable String setterArg) {
+        this.forceAccountName = setterArg;
+        return this;
+      }
+
       public @NonNull InitParams build() {
         InitParams pigeonReturn = new InitParams();
         pigeonReturn.setScopes(scopes);
@@ -240,19 +265,21 @@ public static final class Builder {
         pigeonReturn.setClientId(clientId);
         pigeonReturn.setServerClientId(serverClientId);
         pigeonReturn.setForceCodeForRefreshToken(forceCodeForRefreshToken);
+        pigeonReturn.setForceAccountName(forceAccountName);
         return pigeonReturn;
       }
     }
 
     @NonNull
     ArrayList<Object> toList() {
-      ArrayList<Object> toListResult = new ArrayList<>(6);
+      ArrayList<Object> toListResult = new ArrayList<>(7);
       toListResult.add(scopes);
       toListResult.add(signInType);
       toListResult.add(hostedDomain);
       toListResult.add(clientId);
       toListResult.add(serverClientId);
       toListResult.add(forceCodeForRefreshToken);
+      toListResult.add(forceAccountName);
       return toListResult;
     }
 
@@ -270,6 +297,8 @@ ArrayList<Object> toList() {
       pigeonResult.setServerClientId((String) serverClientId);
       Object forceCodeForRefreshToken = pigeonVar_list.get(5);
       pigeonResult.setForceCodeForRefreshToken((Boolean) forceCodeForRefreshToken);
+      Object forceAccountName = pigeonVar_list.get(6);
+      pigeonResult.setForceAccountName((String) forceAccountName);
       return pigeonResult;
     }
   }
@@ -512,6 +541,7 @@ public interface Result<T> {
     /** Failure case callback method for handling errors. */
     void error(@NonNull Throwable error);
   }
+
   /** Asynchronous error handling return type for nullable API method returns. */
   public interface NullableResult<T> {
     /** Success case callback method for handling returns. */
@@ -520,6 +550,7 @@ public interface NullableResult<T> {
     /** Failure case callback method for handling errors. */
     void error(@NonNull Throwable error);
   }
+
   /** Asynchronous error handling return type for void API method returns. */
   public interface VoidResult {
     /** Success case callback method for handling returns. */
@@ -528,33 +559,43 @@ public interface VoidResult {
     /** Failure case callback method for handling errors. */
     void error(@NonNull Throwable error);
   }
+
   /** Generated interface from Pigeon that represents a handler of messages from Flutter. */
   public interface GoogleSignInApi {
     /** Initializes a sign in request with the given parameters. */
     void init(@NonNull InitParams params);
+
     /** Starts a silent sign in. */
     void signInSilently(@NonNull Result<UserData> result);
+
     /** Starts a sign in with user interaction. */
     void signIn(@NonNull Result<UserData> result);
+
     /** Requests the access token for the current sign in. */
     void getAccessToken(
         @NonNull String email, @NonNull Boolean shouldRecoverAuth, @NonNull Result<String> result);
+
     /** Signs out the current user. */
     void signOut(@NonNull VoidResult result);
+
     /** Revokes scope grants to the application. */
     void disconnect(@NonNull VoidResult result);
+
     /** Returns whether the user is currently signed in. */
     @NonNull
     Boolean isSignedIn();
+
     /** Clears the authentication caching for the given token, requiring a new sign in. */
     void clearAuthCache(@NonNull String token);
+
     /** Requests access to the given scopes. */
     void requestScopes(@NonNull List<String> scopes, @NonNull Result<Boolean> result);
 
     /** The codec used by GoogleSignInApi. */
     static @NonNull MessageCodec<Object> getCodec() {
       return PigeonCodec.INSTANCE;
     }
+
     /**
      * Sets up an instance of `GoogleSignInApi` to handle messages through the `binaryMessenger`.
      */

packages/google_sign_in/google_sign_in_android/android/src/test/java/io/flutter/plugins/googlesignin/GoogleSignInTest.java

@@ -10,6 +10,7 @@
 import static org.mockito.Mockito.verify;
 import static org.mockito.Mockito.when;
 
+import android.accounts.Account;
 import android.app.Activity;
 import android.content.Context;
 import android.content.Intent;
@@ -32,6 +33,8 @@
 import org.junit.Test;
 import org.mockito.ArgumentCaptor;
 import org.mockito.Mock;
+import org.mockito.MockedConstruction;
+import org.mockito.Mockito;
 import org.mockito.MockitoAnnotations;
 import org.mockito.Spy;
 
@@ -284,6 +287,25 @@ public void init_PassesForceCodeForRefreshTokenTrueWithServerClientIdFromResourc
     initAndAssertForceCodeForRefreshToken(params, true);
   }
 
+  @Test
+  public void init_PassesForceAccountName() {
+    String fakeAccountName = "[email protected]";
+
+    try (MockedConstruction<Account> mocked =
+        Mockito.mockConstruction(
+            Account.class,
+            (mock, context) -> {
+              when(mock.toString()).thenReturn(fakeAccountName);
+            })) {
+      InitParams params = buildInitParams("fakeClientId", "fakeServerClientId2", fakeAccountName);
+
+      initAndAssertForceAccountName(params, fakeAccountName);
+
+      List<Account> constructed = mocked.constructed();
+      Assert.assertEquals(1, constructed.size());
+    }
+  }
+
   public void initAndAssertServerClientId(InitParams params, String serverClientId) {
     ArgumentCaptor<GoogleSignInOptions> optionsCaptor =
         ArgumentCaptor.forClass(GoogleSignInOptions.class);
@@ -304,9 +326,23 @@ public void initAndAssertForceCodeForRefreshToken(
         forceCodeForRefreshToken, optionsCaptor.getValue().isForceCodeForRefreshToken());
   }
 
+  public void initAndAssertForceAccountName(InitParams params, String forceAccountName) {
+    ArgumentCaptor<GoogleSignInOptions> optionsCaptor =
+        ArgumentCaptor.forClass(GoogleSignInOptions.class);
+    when(mockGoogleSignIn.getClient(any(Context.class), optionsCaptor.capture()))
+        .thenReturn(mockClient);
+    plugin.init(params);
+    Assert.assertEquals(forceAccountName, optionsCaptor.getValue().getAccount().toString());
+  }
+
   private static InitParams buildInitParams(String clientId, String serverClientId) {
     return buildInitParams(
-        Messages.SignInType.STANDARD, Collections.emptyList(), clientId, serverClientId, false);
+        Messages.SignInType.STANDARD,
+        Collections.emptyList(),
+        clientId,
+        serverClientId,
+        false,
+        null);
   }
 
   private static InitParams buildInitParams(
@@ -316,15 +352,28 @@ private static InitParams buildInitParams(
         Collections.emptyList(),
         clientId,
         serverClientId,
-        forceCodeForRefreshToken);
+        forceCodeForRefreshToken,
+        null);
+  }
+
+  private static InitParams buildInitParams(
+      String clientId, String serverClientId, String forceAccountName) {
+    return buildInitParams(
+        Messages.SignInType.STANDARD,
+        Collections.emptyList(),
+        clientId,
+        serverClientId,
+        false,
+        forceAccountName);
   }
 
   private static InitParams buildInitParams(
       Messages.SignInType signInType,
       List<String> scopes,
       String clientId,
       String serverClientId,
-      boolean forceCodeForRefreshToken) {
+      boolean forceCodeForRefreshToken,
+      String forceAccountName) {
     InitParams.Builder builder = new InitParams.Builder();
     builder.setSignInType(signInType);
     builder.setScopes(scopes);
@@ -335,6 +384,9 @@ private static InitParams buildInitParams(
       builder.setServerClientId(serverClientId);
     }
     builder.setForceCodeForRefreshToken(forceCodeForRefreshToken);
+    if (forceAccountName != null) {
+      builder.setForceAccountName(forceAccountName);
+    }
     return builder.build();
   }
 }

packages/google_sign_in/google_sign_in_android/example/pubspec.yaml

@@ -16,7 +16,7 @@ dependencies:
     # The example app is bundled with the plugin so we use a path dependency on
     # the parent directory to use the current plugin's version.
     path: ../
-  google_sign_in_platform_interface: ^2.2.0
+  google_sign_in_platform_interface: ^2.5.0
   http: ">=0.13.0 <2.0.0"
 
 dev_dependencies:

packages/google_sign_in/google_sign_in_android/lib/google_sign_in_android.dart

@@ -29,12 +29,14 @@ class GoogleSignInAndroid extends GoogleSignInPlatform {
     SignInOption signInOption = SignInOption.standard,
     String? hostedDomain,
     String? clientId,
+    String? forceAccountName,
   }) {
     return initWithParams(SignInInitParameters(
       signInOption: signInOption,
       scopes: scopes,
       hostedDomain: hostedDomain,
       clientId: clientId,
+      forceAccountName: forceAccountName,
     ));
   }
 
@@ -47,6 +49,7 @@ class GoogleSignInAndroid extends GoogleSignInPlatform {
       clientId: params.clientId,
       serverClientId: params.serverClientId,
       forceCodeForRefreshToken: params.forceCodeForRefreshToken,
+      forceAccountName: params.forceAccountName,
     ));
   }
 

packages/google_sign_in/google_sign_in_android/lib/src/messages.g.dart

@@ -38,6 +38,7 @@ class InitParams {
     this.clientId,
     this.serverClientId,
     this.forceCodeForRefreshToken = false,
+    this.forceAccountName,
   });
 
   List<String> scopes;
@@ -52,6 +53,8 @@ class InitParams {
 
   bool forceCodeForRefreshToken;
 
+  String? forceAccountName;
+
   Object encode() {
     return <Object?>[
       scopes,
@@ -60,6 +63,7 @@ class InitParams {
       clientId,
       serverClientId,
       forceCodeForRefreshToken,
+      forceAccountName,
     ];
   }
 
@@ -72,6 +76,7 @@ class InitParams {
       clientId: result[3] as String?,
       serverClientId: result[4] as String?,
       forceCodeForRefreshToken: result[5]! as bool,
+      forceAccountName: result[6] as String?,
     );
   }
 }

packages/google_sign_in/google_sign_in_android/pigeons/messages.dart

@@ -33,6 +33,7 @@ class InitParams {
     this.clientId,
     this.serverClientId,
     this.forceCodeForRefreshToken = false,
+    this.forceAccountName,
   });
 
   final List<String> scopes;
@@ -41,6 +42,7 @@ class InitParams {
   final String? clientId;
   final String? serverClientId;
   final bool forceCodeForRefreshToken;
+  final String? forceAccountName;
 }
 
 /// Pigeon version of GoogleSignInUserData.

packages/google_sign_in/google_sign_in_android/pubspec.yaml

@@ -2,7 +2,7 @@ name: google_sign_in_android
 description: Android implementation of the google_sign_in plugin.
 repository: https://github.com/flutter/packages/tree/main/packages/google_sign_in/google_sign_in_android
 issue_tracker: https://github.com/flutter/flutter/issues?q=is%3Aissue+is%3Aopen+label%3A%22p%3A+google_sign_in%22
-version: 6.1.36
+version: 6.2.0
 
 environment:
   sdk: ^3.6.0
@@ -20,7 +20,7 @@ flutter:
 dependencies:
   flutter:
     sdk: flutter
-  google_sign_in_platform_interface: ^2.2.0
+  google_sign_in_platform_interface: ^2.5.0
 
 dev_dependencies:
   build_runner: ^2.3.0

packages/google_sign_in/google_sign_in_android/test/google_sign_in_android_test.dart

@@ -143,6 +143,7 @@ void main() {
       clientId: 'fakeClientId',
       serverClientId: 'fakeServerClientId',
       forceCodeForRefreshToken: true,
+      forceAccountName: '[email protected]',
     );
 
     await googleSignIn.initWithParams(initParams);
@@ -156,6 +157,7 @@ void main() {
     expect(passedParams.serverClientId, initParams.serverClientId);
     expect(passedParams.forceCodeForRefreshToken,
         initParams.forceCodeForRefreshToken);
+    expect(passedParams.forceAccountName, initParams.forceAccountName);
   });
 
   test('clearAuthCache passes arguments', () async {