[google_sign_in_web] Migrate to null-safety (#3628)

Author: ditman

packages/google_sign_in/google_sign_in_web/CHANGELOG.md

@@ -1,3 +1,7 @@
+## 0.10.0
+
+* Migrate to null-safety.
+
 ## 0.9.2+1
 
 * Update Flutter SDK constraint.

packages/google_sign_in/google_sign_in_web/example/README.md

@@ -0,0 +1,21 @@
+# Testing
+
+This package utilizes the `integration_test` package to run its tests in a web browser.
+
+See [flutter.dev > Integration testing](https://flutter.dev/docs/testing/integration-tests) for more info.
+
+## Running the tests
+
+Make sure you have updated to the latest Flutter master.
+
+1. Check what version of Chrome is running on the machine you're running tests on.
+
+2. Download and install driver for that version from here:
+    * <https://chromedriver.chromium.org/downloads>
+
+3. Start the driver using `chromedriver --port=4444`
+
+4. Run tests: `flutter drive -d web-server --browser-name=chrome --driver=test_driver/integration_driver.dart --target=integration_test/TEST_NAME.dart`, or (in Linux):
+
+    * Single: `./run_test.sh integration_test/TEST_NAME.dart`
+    * All: `./run_test.sh`

packages/google_sign_in/google_sign_in_web/test/test_driver/auth2_integration.dart renamed to packages/google_sign_in/google_sign_in_web/example/integration_test/auth2_test.dart

@@ -3,11 +3,12 @@
 // found in the LICENSE file.
 
 import 'package:flutter/services.dart';
-import 'package:integration_test/integration_test.dart';
-
 import 'package:flutter_test/flutter_test.dart';
 import 'package:google_sign_in_platform_interface/google_sign_in_platform_interface.dart';
 import 'package:google_sign_in_web/google_sign_in_web.dart';
+import 'package:integration_test/integration_test.dart';
+import 'package:js/js_util.dart' as js_util;
+
 import 'gapi_mocks/gapi_mocks.dart' as gapi_mocks;
 import 'src/test_utils.dart';
 
@@ -25,7 +26,7 @@ void main() {
     idToken: expectedTokenData.idToken,
   );
 
-  GoogleSignInPlugin plugin;
+  late GoogleSignInPlugin plugin;
 
   group('plugin.init() throws a catchable exception', () {
     setUp(() {
@@ -54,15 +55,16 @@ void main() {
         );
         fail('plugin.init should have thrown an exception!');
       } catch (e) {
-        expect(e.code, 'idpiframe_initialization_failed');
+        final String code = js_util.getProperty(e, 'code') as String;
+        expect(code, 'idpiframe_initialization_failed');
       }
     });
   });
 
   group('other methods also throw catchable exceptions on init fail', () {
     // This function ensures that init gets called, but for some reason, we
     // ignored that it has thrown stuff...
-    void _discardInit() async {
+    Future<void> _discardInit() async {
       try {
         await plugin.init(
           hostedDomain: 'foo',
@@ -135,13 +137,13 @@ void main() {
       });
 
       testWidgets('signInSilently', (WidgetTester tester) async {
-        GoogleSignInUserData actualUser = await plugin.signInSilently();
+        GoogleSignInUserData actualUser = (await plugin.signInSilently())!;
 
         expect(actualUser, expectedUserData);
       });
 
       testWidgets('signIn', (WidgetTester tester) async {
-        GoogleSignInUserData actualUser = await plugin.signIn();
+        GoogleSignInUserData actualUser = (await plugin.signIn())!;
 
         expect(actualUser, expectedUserData);
       });
@@ -185,7 +187,8 @@ void main() {
         await plugin.signIn();
         fail('plugin.signIn() should have thrown an exception!');
       } catch (e) {
-        expect(e.code, 'popup_closed_by_user');
+        final String code = js_util.getProperty(e, 'code') as String;
+        expect(code, 'popup_closed_by_user');
       }
     });
   });

packages/google_sign_in/google_sign_in_web/test/test_driver/gapi_load_integration.dart renamed to packages/google_sign_in/google_sign_in_web/example/integration_test/gapi_load_test.dart

@@ -4,18 +4,19 @@
 
 import 'dart:html' as html;
 
-import 'package:integration_test/integration_test.dart';
-
 import 'package:flutter_test/flutter_test.dart';
 import 'package:google_sign_in_platform_interface/google_sign_in_platform_interface.dart';
 import 'package:google_sign_in_web/google_sign_in_web.dart';
+import 'package:integration_test/integration_test.dart';
+
 import 'gapi_mocks/gapi_mocks.dart' as gapi_mocks;
 import 'src/test_utils.dart';
 
 void main() {
   IntegrationTestWidgetsFlutterBinding.ensureInitialized();
 
-  gapiUrl = toBase64Url(gapi_mocks.auth2InitSuccess(GoogleSignInUserData()));
+  gapiUrl = toBase64Url(gapi_mocks.auth2InitSuccess(
+      GoogleSignInUserData(email: '[email protected]', id: '1234')));
 
   testWidgets('Plugin is initialized after GAPI fully loads and init is called',
       (WidgetTester tester) async {

packages/google_sign_in/google_sign_in_web/test/test_driver/gapi_mocks/gapi_mocks.dart renamed to packages/google_sign_in/google_sign_in_web/example/integration_test/gapi_mocks/gapi_mocks.dart

@@ -1,27 +1,21 @@
 // Copyright 2019 The Chromium 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 'package:flutter_test/flutter_test.dart';
-
-import 'package:integration_test/integration_test.dart';
 
+import 'package:flutter_test/flutter_test.dart';
 import 'package:google_sign_in_web/src/generated/gapiauth2.dart' as gapi;
 import 'package:google_sign_in_web/src/utils.dart';
-import 'package:mockito/mockito.dart';
-
-class MockGoogleUser extends Mock implements gapi.GoogleUser {}
-
-class MockBasicProfile extends Mock implements gapi.BasicProfile {}
+import 'package:integration_test/integration_test.dart';
 
 void main() {
   // The non-null use cases are covered by the auth2_test.dart file.
   IntegrationTestWidgetsFlutterBinding.ensureInitialized();
 
   group('gapiUserToPluginUserData', () {
-    var mockUser;
+    late FakeGoogleUser fakeUser;
 
     setUp(() {
-      mockUser = MockGoogleUser();
+      fakeUser = FakeGoogleUser();
     });
 
     testWidgets('null user -> null response', (WidgetTester tester) async {
@@ -30,21 +24,45 @@ void main() {
 
     testWidgets('not signed-in user -> null response',
         (WidgetTester tester) async {
-      when(mockUser.isSignedIn()).thenReturn(false);
-      expect(gapiUserToPluginUserData(mockUser), isNull);
+      expect(gapiUserToPluginUserData(fakeUser), isNull);
     });
 
     testWidgets('signed-in, but null profile user -> null response',
         (WidgetTester tester) async {
-      when(mockUser.isSignedIn()).thenReturn(true);
-      expect(gapiUserToPluginUserData(mockUser), isNull);
+      fakeUser.setIsSignedIn(true);
+      expect(gapiUserToPluginUserData(fakeUser), isNull);
     });
 
     testWidgets('signed-in, null userId in profile user -> null response',
         (WidgetTester tester) async {
-      when(mockUser.isSignedIn()).thenReturn(true);
-      when(mockUser.getBasicProfile()).thenReturn(MockBasicProfile());
-      expect(gapiUserToPluginUserData(mockUser), isNull);
+      fakeUser.setIsSignedIn(true);
+      fakeUser.setBasicProfile(FakeBasicProfile());
+      expect(gapiUserToPluginUserData(fakeUser), isNull);
     });
   });
 }
+
+class FakeGoogleUser extends Fake implements gapi.GoogleUser {
+  bool _isSignedIn = false;
+  gapi.BasicProfile? _basicProfile;
+
+  @override
+  bool isSignedIn() => _isSignedIn;
+  @override
+  gapi.BasicProfile? getBasicProfile() => _basicProfile;
+
+  void setIsSignedIn(bool isSignedIn) {
+    _isSignedIn = isSignedIn;
+  }
+
+  void setBasicProfile(gapi.BasicProfile basicProfile) {
+    _basicProfile = basicProfile;
+  }
+}
+
+class FakeBasicProfile extends Fake implements gapi.BasicProfile {
+  String? _id;
+
+  @override
+  String? getId() => _id;
+}

packages/google_sign_in/google_sign_in_web/test/test_driver/gapi_mocks/src/auth2_init.dart renamed to packages/google_sign_in/google_sign_in_web/example/integration_test/gapi_mocks/src/auth2_init.dart

@@ -1,23 +1,23 @@
-name: regular_integration_tests
+name: google_sign_in_web_integration_tests
 publish_to: none
 
 environment:
-  sdk: ">=2.2.2 <3.0.0"
+  sdk: ">=2.12.0-259.9.beta <3.0.0"
+  flutter: ">=1.27.0-0" # For integration_test from sdk
 
 dependencies:
   flutter:
     sdk: flutter
 
 dev_dependencies:
-  google_sign_in: ^4.5.3
+  http: ^0.13.0
+  js: ^0.6.3
   flutter_driver:
     sdk: flutter
   flutter_test:
     sdk: flutter
-  http: ^0.12.2
-  mockito: ^4.1.1
   integration_test:
-    path: ../../../integration_test
+    sdk: flutter
 
 dependency_overrides:
   google_sign_in_web:

packages/google_sign_in/google_sign_in_web/test/test_driver/gapi_mocks/src/gapi.dart renamed to packages/google_sign_in/google_sign_in_web/example/integration_test/gapi_mocks/src/gapi.dart

@@ -1,17 +1,20 @@
 #!/usr/bin/bash
+
 if pgrep -lf chromedriver > /dev/null; then
   echo "chromedriver is running."
 
   if [ $# -eq 0 ]; then
     echo "No target specified, running all tests..."
-    find test_driver/ -iname *_integration.dart | xargs -n1 -i -t flutter drive -d web-server --web-port=7357 --browser-name=chrome --target='{}'
+    find integration_test/ -iname *_test.dart | xargs -n1 -i -t flutter drive -d web-server --web-port=7357 --browser-name=chrome  --driver=test_driver/integration_driver.dart --target='{}'
   else
     echo "Running test target: $1..."
     set -x
-    flutter drive -d web-server --web-port=7357 --browser-name=chrome --target=$1
+    flutter drive -d web-server --web-port=7357 --browser-name=chrome --driver=test_driver/integration_driver.dart --target=$1
   fi
 
   else
     echo "chromedriver is not running."
 fi
 
+
+