Address PR comments / checks.

Author: ditman

packages/google_sign_in/google_sign_in_web/example/integration_test/src/create_jwt.dart

@@ -0,0 +1,34 @@
+// Copyright 2013 The Flutter Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+import 'dart:convert';
+
+import 'package:google_identity_services_web/id.dart';
+import 'package:jose/jose.dart';
+
+import 'jsify_as.dart';
+
+/// Wraps a key-value map of [claims] in a JWT token similar GIS'.
+///
+/// Note that the encryption of this token is weak, and this method should
+/// only be used for tests!
+CredentialResponse createJwt(Map<String, Object?>? claims) {
+  String? credential;
+  if (claims != null) {
+    final JsonWebTokenClaims token = JsonWebTokenClaims.fromJson(claims);
+    final JsonWebSignatureBuilder builder = JsonWebSignatureBuilder();
+    builder.jsonContent = token.toJson();
+    builder.addRecipient(
+        JsonWebKey.fromJson(<String, Object?>{
+          'kty': 'oct',
+          'k': base64.encode('symmetric-encryption-is-weak'.codeUnits),
+        }),
+        algorithm: 'HS256'); // bogus crypto, don't use this for prod!
+    builder.setProtectedHeader('typ', 'JWT');
+    credential = builder.build().toCompactSerialization();
+  }
+  return jsifyAs<CredentialResponse>(<String, Object?>{
+    'credential': credential,
+  });
+}

packages/google_sign_in/google_sign_in_web/example/integration_test/utils_test.dart

@@ -2,16 +2,14 @@
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
-import 'dart:convert';
-
 import 'package:flutter_test/flutter_test.dart';
 import 'package:google_identity_services_web/id.dart';
 import 'package:google_identity_services_web/oauth2.dart';
 import 'package:google_sign_in_platform_interface/google_sign_in_platform_interface.dart';
 import 'package:google_sign_in_web/src/utils.dart';
 import 'package:integration_test/integration_test.dart';
-import 'package:jose/jose.dart';
 
+import 'src/create_jwt.dart';
 import 'src/jsify_as.dart';
 
 void main() {
@@ -80,25 +78,3 @@ void main() {
     });
   });
 }
-
-CredentialResponse createJwt(Map<String, Object?>? claims) {
-  String? credential;
-  if (claims != null) {
-    final JsonWebTokenClaims token = JsonWebTokenClaims.fromJson(claims);
-    final JsonWebSignatureBuilder builder = JsonWebSignatureBuilder();
-    builder.jsonContent = token.toJson();
-    builder.addRecipient(
-        JsonWebKey.fromJson(<String, Object?>{
-          'kty': 'oct',
-          'k': base64.encode('symmetric-encryption-is-weak'.codeUnits),
-        }),
-        algorithm: 'HS256'); // bogus crypto, don't use this for prod!
-    builder.setProtectedHeader('typ', 'JWT');
-    credential = builder.build().toCompactSerialization();
-    // ignore:avoid_print
-    print('Generated JWT: $credential');
-  }
-  return jsifyAs<CredentialResponse>(<String, Object?>{
-    'credential': credential,
-  });
-}

packages/google_sign_in/google_sign_in_web/lib/src/gis_client.dart

@@ -199,7 +199,7 @@ class GisSdkClient {
     // If we already know the user, use their `email` as a `hint`, so they don't
     // have to pick their user again in the Authorization popup.
     final GoogleSignInUserData? knownUser =
-      utils.gisResponsesToUserData(_lastCredentialResponse);
+        utils.gisResponsesToUserData(_lastCredentialResponse);
     // This toggles a popup, so `signIn` *must* be called with
     // user activation.
     _tokenClient.requestAccessToken(OverridableTokenClientConfig(
@@ -287,7 +287,7 @@ class GisSdkClient {
 
   // The scopes initially requested by the developer.
   //
-  // We store this because we might need to add more at `signIn`, if the user
+  // We store this because we might need to add more at `signIn`. If the user
   // doesn't `silentSignIn`, we expand this list to consult the People API to
   // return some basic Authentication information.
   final List<String> _initialScopes;